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

CompatHelper: bump compat for OptimizationOptimJL to 0.2, (keep existing compat) #175

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jan 6, 2024

This pull request changes the compat entry for the OptimizationOptimJL package from 0.1 to 0.1, 0.2.
This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.

@github-actions github-actions bot force-pushed the compathelper/new_version/2024-01-06-00-14-59-543-04204179526 branch from b30ff14 to f165fa2 Compare January 6, 2024 00:15
@sethaxen sethaxen closed this Jan 6, 2024
@sethaxen sethaxen reopened this Jan 6, 2024
Copy link

codecov bot commented Jan 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (26641dc) 92.92% compared to head (e96de93) 93.10%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #175      +/-   ##
==========================================
+ Coverage   92.92%   93.10%   +0.17%     
==========================================
  Files          13       13              
  Lines         594      609      +15     
==========================================
+ Hits          552      567      +15     
  Misses         42       42              

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

@sethaxen
Copy link
Member

sethaxen commented Jan 6, 2024

There are some breaking changes to the callback arguments we need to adjust for, see SciML/Optimization.jl#652 (comment)

@sethaxen
Copy link
Member

It seems to support OptimizationOptimJL, we would need Optimization v3.21.0, which would require us to drop support for Julia v1.6-v1.8, see SciML/Optimization.jl#639

@sethaxen
Copy link
Member

By conditioning on the existence of Optimization.OptimizationState, we can support older versions of Optimization.jl for older Julia versions, so this now works.

Now that v1.10 is released, this is redundant.
@sethaxen sethaxen merged commit c5990eb into main Jan 15, 2024
10 of 11 checks passed
@sethaxen sethaxen deleted the compathelper/new_version/2024-01-06-00-14-59-543-04204179526 branch January 15, 2024 12:21
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.

1 participant