Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Propagation improvements #140

Merged
merged 7 commits into from Sep 26, 2016

Conversation

astrojuanlu added 3 commits Sep 11, 2016
The callback is called at each internal integrator step, therefore
enabling the user to retrieve every intermediate state.
@codecov-io
Copy link

@codecov-io codecov-io commented Sep 11, 2016

Current coverage is 82.15% (diff: 100%)

Merging #140 into master will increase coverage by 0.48%

@@             master       #140   diff @@
==========================================
  Files            23         23          
  Lines           906        908     +2   
  Methods           0          0          
  Messages          0          0          
  Branches         71         72     +1   
==========================================
+ Hits            740        746     +6   
+ Misses          145        142     -3   
+ Partials         21         20     -1   

Powered by Codecov. Last update 573212e...ed931ad

astrojuanlu added 4 commits Sep 13, 2016
This leads to huge performance improvements, as `quantity_input` is very slow.
Still extremely slow.
One untouched test still failing
@astrojuanlu
Copy link
Member Author

@astrojuanlu astrojuanlu commented Sep 26, 2016

This PR has been standing for too long. Changes in plotting code can come later.

@astrojuanlu astrojuanlu merged commit 023c114 into poliastro:master Sep 26, 2016
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
codeclimate Code Climate didn't find any new or fixed issues.
Details
@astrojuanlu astrojuanlu deleted the astrojuanlu:propagation-improvements branch Sep 26, 2016
@astrojuanlu astrojuanlu mentioned this pull request Mar 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants