Menu ▾ ▴

#701 Let taylor return coefficients

v1.1.x
closed
Erik Hänel
internal (109) cmd (19)
v1.1.4
New Functionality
2022-08-05
2022-05-14
Erik Hänel
No

Change the taylor command to actually return the calculated coefficients.

Analysis:

That's doable by switching the command type, adding the coefficients to a std::vector and set that as return value. It might be necessary to change the return type, however, it is always needed that the return value of the command handler is adapted and that the documentation is updated.

Implementation:

  • Implementation: Implemented as described by the analysis.
  • Revision: [r1156]
  • Implementation test: Command was used and returned the calculated coefficients.

Documentation:

  • [x] ChangesLog updated
  • [x] Code changes commented
  • Documentation articles:
    • [x] corresponding documentation articles updated
    • [ ] new documentation articles created
    • [ ] not needed
  • Language files:
    • [x] corresponding language files updated
    • [ ] not needed

Tests:

taylor is part of the automatic SW tests. No deviations detected.

Related

Commit: [r1156]

Discussion

  • Erik Hänel

    Erik Hänel - 2022-05-25
    • labels: --> internal, cmd
    • status: open --> accepted
     

  • Erik Hänel

    Erik Hänel - 2022-05-25
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1 +1,23 @@
 Change the `taylor` command to actually return the calculated coefficients.
+
+###Analysis:
+(*Describe, what's the issue and which changes have to be made*)
+
+###Implementation:
+* Implementation: (*Describe, what you've changed*) 
+* Revision: [rXXX]
+* Implementation test: (*Describe the type of test, which you performed, and if it was successful*)
+
+###Documentation:
+* [ ] ChangesLog updated
+* [ ] Code changes commented
+* **Documentation articles:**

+    * [ ] corresponding documentation articles updated
+    * [ ] new documentation articles created
+    * [ ] not needed
+* **Language files:**
+    * [ ] corresponding language files updated
+    * [ ] not needed
+
+###Tests:
+(*Describe, which tests you performed and their outcome*)
    • status: accepted --> analyzing
     

  • Erik Hänel

    Erik Hänel - 2022-05-27
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1,7 +1,7 @@
 Change the `taylor` command to actually return the calculated coefficients.

 ###Analysis:
-(*Describe, what's the issue and which changes have to be made*)
+That's doable by switching the command type, adding the coefficients to a `std::vector` and set that as return value. It might be necessary to change the return type, however, it is always needed that the return value of the command handler is adapted and that the documentation is updated.

 ###Implementation:

 * Implementation: (*Describe, what you've changed*)
    • status: analyzing --> implementing
     

  • Erik Hänel

    Erik Hänel - 2022-05-27
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -4,19 +4,19 @@
 That's doable by switching the command type, adding the coefficients to a `std::vector` and set that as return value. It might be necessary to change the return type, however, it is always needed that the return value of the command handler is adapted and that the documentation is updated.

 ###Implementation:
-* Implementation: (*Describe, what you've changed*) 
-* Revision: [rXXX]
-* Implementation test: (*Describe the type of test, which you performed, and if it was successful*)
+* Implementation: Implemented as described by the analysis.
+* Revision: [r1156]
+* Implementation test: Command was used and returned the calculated coefficients.

 ###Documentation:
-* [ ] ChangesLog updated
-* [ ] Code changes commented
+* [x] ChangesLog updated
+* [x] Code changes commented

 * **Documentation articles:**
-    * [ ] corresponding documentation articles updated
+    * [x] corresponding documentation articles updated
     * [ ] new documentation articles created
     * [ ] not needed
 * **Language files:**
-    * [ ] corresponding language files updated
+    * [x] corresponding language files updated
     * [ ] not needed

 ###Tests:
    • status: implementing --> testing
     

    Related

    Commit: [r1156]

  • Erik Hänel

    Erik Hänel - 2022-08-05
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -20,4 +20,4 @@

     * [ ] not needed

 ###Tests:
-(*Describe, which tests you performed and their outcome*)
+`taylor` is part of the automatic SW tests. No deviations detected.
    • status: testing --> closed
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB