-
Notifications
You must be signed in to change notification settings - Fork 24
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
Adds plotting methods #115
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #115 +/- ##
===========================================
+ Coverage 90.57% 92.65% +2.07%
===========================================
Files 18 30 +12
Lines 647 1007 +360
===========================================
+ Hits 586 933 +347
- Misses 61 74 +13 ☔ View full report in Codecov by Sentry. |
… class, Updt example
…hange plot_convergence() to plot mininum cost trace, merge develop
This should now be ready for review. Four methods have been added:
All the plotting is completed using plotly, with the setup.py updated to have an optional installation of See updated initial comment on functionality with nlopt and scipy. Here are some sample figures, generated from the
and with (where the red marker is the starting position): |
…g and readme for development installation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work on these plotting scripts, thanks @BradyPlanden! The plots are clear and useful - a great addition to PyBOP.
…nstallation, updt. quick_plot and plot_parameter
This PR completes the following:
Update:
I've left the slider off this PR as future functionality to add. Additionally, the above plots work across all optimisers, except
plot_parameters()
andplot_convergance()
which don't currently support NLopt and SciPy optimisers.