Hi. I just committed the changes we talked about. Now @i is first in both sums and products.
I have added/modified match patterns (Complex a b) which matches and binds a to the real part and b to the imaginary part. Also the match pattern (ImaginaryTerm u) matches only imaginary terms.
I also changed the implementation of Angle slightly. There were some cases that the previous test cases didn't catch. Now tests for all four quadrants are in place. The change fixed a problem in Ln too.
Some/all of your pull requests now say "conflicts must be resolved". Sorry about that. I think this commit simply touched large parts of the code.
I left the comment the wrong place:
Hi. I just committed the changes we talked about. Now @i is first in both sums and products. I have added/modified match patterns (Complex a b) which matches and binds a to the real part and b to the imaginary part. Also the match pattern (ImaginaryTerm u) matches only imaginary terms.
I also changed the implementation of Angle slightly. There were some cases that the previous test cases didn't catch. Now tests for all four quadrants are in place. The change fixed a problem in Ln too.
Some/all of your pull requests now say "conflicts must be resolved". Sorry about that. I think this commit simply touched large parts of the code.
/Jens Axel