-
Notifications
You must be signed in to change notification settings - Fork 57
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
Refactor diffeq #179
Refactor diffeq #179
Conversation
Codecov Report
@@ Coverage Diff @@
## master #179 +/- ##
==========================================
- Coverage 73.19% 73.18% -0.02%
==========================================
Files 58 58
Lines 3026 3024 -2
Branches 391 391
==========================================
- Hits 2215 2213 -2
Misses 660 660
Partials 151 151
Continue to review full report at Codecov.
|
@pnkraemer Did you use admin privileges to merge this? Currently |
not that I know of, but in any case black did not complain about this PR. |
This could again be due to a new Black release. I'll fix it by pinning the version. |
steprule is now an argument for solve(), not an attribute of the solver. This is the first step of the refactoring, but independent enough that it can be merged independently.