-
Notifications
You must be signed in to change notification settings - Fork 47
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
support for NLopt 1.0 #706
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #706 +/- ##
==========================================
- Coverage 95.83% 95.83% -0.01%
==========================================
Files 35 35
Lines 3266 3263 -3
==========================================
- Hits 3130 3127 -3
Misses 136 136
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
I may have done the approvals of PR's backwards. I approved and committed the PR that bumps the version to 4.18.0 whereas this PR bumps the version to 4.17.1. Should that be changed now to 4.18.1? I haven't alerted the JuliaRegistrator to the version bump to 4.18.0 |
Update version to 4.18.1, not 4.17.1
@dmbates we can just roll them together into a single release as 4.18.0. |
This bumps NLopt support for the 1.0 release, which moves MathOptInterface support to an extension. This dramatically improves load times for MixedModels, which doesn't need MathOptInterface. I've dropped support for NLopt 0.5 because the callbacks don't work on Julia 1.5+ (due to the stack allocation of immutable structs -- the struct was changed to mutable in 0.6), since we're already compat bounded to Julia 1.8.