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

Update plotting capabilities and dataset restructure #178

Merged
merged 85 commits into from
Mar 21, 2024

Fix version check

e513c33
Select commit
Loading
Failed to load commit list.
Merged

Update plotting capabilities and dataset restructure #178

Fix version check
e513c33
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Mar 21, 2024 in 0s

95.13% (+0.83%) compared to 2013161

View this Pull Request on Codecov

95.13% (+0.83%) compared to 2013161

Details

Codecov Report

Attention: Patch coverage is 95.97701% with 21 lines in your changes are missing coverage. Please review.

Project coverage is 95.13%. Comparing base (2013161) to head (e513c33).
Report is 6 commits behind head on develop.

Files Patch % Lines
pybop/costs/_likelihoods.py 91.22% 5 Missing ⚠️
pybop/plotting/quick_plot.py 95.68% 5 Missing ⚠️
pybop/costs/fitting_costs.py 93.47% 3 Missing ⚠️
pybop/observers/observer.py 90.00% 3 Missing ⚠️
pybop/models/lithium_ion/echem_base.py 50.00% 2 Missing ⚠️
pybop/costs/design_costs.py 95.23% 1 Missing ⚠️
pybop/plotting/plot_problem.py 97.22% 1 Missing ⚠️
pybop/plotting/plotly_manager.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #178      +/-   ##
===========================================
+ Coverage    94.29%   95.13%   +0.83%     
===========================================
  Files           33       36       +3     
  Lines         1753     1994     +241     
===========================================
+ Hits          1653     1897     +244     
+ Misses         100       97       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.