Skip to content

extend arithmetic functions to support differential quantities#127

Open
rjnrohit wants to merge 1 commit intoBoostGSoC19:developfrom
rjnrohit:differential
Open

extend arithmetic functions to support differential quantities#127
rjnrohit wants to merge 1 commit intoBoostGSoC19:developfrom
rjnrohit:differential

Conversation

@rjnrohit
Copy link
Copy Markdown

@rjnrohit rjnrohit commented Apr 14, 2020

Description

  1. Extended arithmetic functions to support differential quantities as well.
  2. Removed magnitude and unit_vector member function from base_differential.hpp
  3. Test for the above implementations has been added.

References

Fixes #125

@rjnrohit rjnrohit changed the title extend arithmetic functions to support for differential quantities extend arithmetic functions to support differential quantities Apr 14, 2020
@rjnrohit
Copy link
Copy Markdown
Author

@sarthak2007 I have created a separate PR as you suggested. you may wanna review it one more time.

Comment thread include/boost/astronomy/coordinate/base_differential.hpp Outdated
@rjnrohit rjnrohit requested a review from sarthak2007 April 14, 2020 16:42
@rjnrohit
Copy link
Copy Markdown
Author

@sarthak2007 I have changed what you had requested. Now there is no conflict of differential argument get passed through make_cartesian_representation. Please have a look at the changes.

Comment thread include/boost/astronomy/coordinate/arithmetic.hpp Outdated
@rjnrohit
Copy link
Copy Markdown
Author

rjnrohit commented Apr 23, 2020

@sarthak2007 changes done! as you have suggested.@lpranam, please see this PR and provide your suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect results are generating with unit_vector() and magnitude() member functions of differential representations.

2 participants