-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add SciPyDifferentialEvolution optimiser #132
Add SciPyDifferentialEvolution optimiser #132
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #132 +/- ##
===========================================
+ Coverage 92.59% 93.03% +0.44%
===========================================
Files 33 34 +1
Lines 1121 1178 +57
===========================================
+ Hits 1038 1096 +58
+ Misses 83 82 -1 ☔ View full report in Codecov by Sentry. |
Hi @BradyPlanden, I added logging of the SciPy iterations to bring this branch up to speed with develop - this is the cause of the conflicts. |
…, baseoptimise, restore test_spm_optimisers assertions
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.
Great work @BradyPlanden! Just one comment on the changelog and then all good to merge, thanks!
This PR adds:
SciPyDifferentialEvolution
optimisermaxiter
as argument to BaseOptimiser with functionality added toNLoptOptimize
andSciPyMinimize