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

Refactor diffeq #179

Merged
merged 5 commits into from
Aug 26, 2020
Merged

Conversation

pnkraemer
Copy link
Collaborator

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.

@codecov
Copy link

codecov bot commented Aug 26, 2020

Codecov Report

Merging #179 into master will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            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              
Impacted Files Coverage Δ
src/probnum/diffeq/odesolver.py 100.00% <ø> (ø)
src/probnum/diffeq/odefiltsmooth/ivpfiltsmooth.py 93.05% <100.00%> (-0.10%) ⬇️
src/probnum/diffeq/odefiltsmooth/odefiltsmooth.py 69.00% <100.00%> (+0.31%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 146037c...fb526a9. Read the comment docs.

@pnkraemer pnkraemer mentioned this pull request Aug 26, 2020
@pnkraemer pnkraemer linked an issue Aug 26, 2020 that may be closed by this pull request
@pnkraemer pnkraemer merged commit 0b85017 into probabilistic-numerics:master Aug 26, 2020
@JonathanWenger
Copy link
Contributor

JonathanWenger commented Aug 26, 2020

@pnkraemer Did you use admin privileges to merge this? Currently black is failing on master.

@pnkraemer
Copy link
Collaborator Author

not that I know of, but in any case black did not complain about this PR.

@nathanaelbosch
Copy link
Collaborator

This could again be due to a new Black release. I'll fix it by pinning the version.

@pnkraemer pnkraemer deleted the refactor_diffeq branch November 13, 2020 08:20
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.

code coverage fail
3 participants