Menu

#214 Assignment to temporary vectors not possible in matop mode

v1.1.x
fixed
None
v1.1.0
Bug Fix
2017-08-20
2017-06-26
Erik Hänel
No

The matop mode allows the assignment of complete vectors and tables, however it is not possible to assign the values to a temporary vector, just like

matop {VAR1,VAR2,VAR3} = one(3,1)

The usage of temporary vectors inside of expressions is however possible.

Analysis:
This fix can be done, in a simple change of the sub matrix evaluation routine, where the target vector has to be considered by shifting the starting point of the for loop. In addition, the sub matrix evaluation routine has to be called every time, which means that the condition checking for matrix functions or matrix multiplications has to be removed.

Implementation:
The fix was implemented as proposed by the analysis. Implementation tests were passed successfully.

Documentation:
Not needed - Bug fix.

Tests:
Software tests were passed without any deviation. Fix implemented successfully.

Discussion

  • Erik Hänel

    Erik Hänel - 2017-06-26
    • status: open --> accepted
     
  • Erik Hänel

    Erik Hänel - 2017-06-26
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -3,3 +3,11 @@
     matop {VAR1,VAR2,VAR3} = one(3,1)
     ~~~
     The usage of temporary vectors inside of expressions is however possible.
    +
    +**Analysis:**
    +
    +**Implementation:**
    +
    +**Documentation:**
    +
    +**Tests:**
    
    • status: accepted --> analyzing
     
  • Erik Hänel

    Erik Hänel - 2017-06-28
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -5,6 +5,7 @@
     The usage of temporary vectors inside of expressions is however possible.
    
     **Analysis:**
    +This fix can be done, in a simple change of the sub matrix evaluation routine, where the target vector has to be considered by shifting the starting point of the for loop. In addition, the sub matrix evaluation routine has to be called every time, which means that the condition checking for matrix functions or matrix multiplications has to be removed.
    
     **Implementation:**
    
    • status: analyzing --> implementing
     
  • Erik Hänel

    Erik Hänel - 2017-06-28
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -8,7 +8,9 @@
     This fix can be done, in a simple change of the sub matrix evaluation routine, where the target vector has to be considered by shifting the starting point of the for loop. In addition, the sub matrix evaluation routine has to be called every time, which means that the condition checking for matrix functions or matrix multiplications has to be removed.
    
     **Implementation:**
    +The fix was implemented as proposed by the analysis. Implementation tests were passed successfully.
    
     **Documentation:**
    +Not needed - Bug fix.
    
     **Tests:**
    
    • status: implementing --> testing
     
  • Erik Hänel

    Erik Hänel - 2017-08-20
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -14,3 +14,4 @@
     Not needed - Bug fix.
    
     **Tests:**
    +Software tests were passed without any deviation. Fix implemented successfully.
    
    • status: testing --> fixed
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB