-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Allow 1.9 #34
Allow 1.9 #34
Conversation
This would need a ton of downstream testing. It's not a trivial change to make. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #34 +/- ##
==========================================
+ Coverage 32.59% 33.14% +0.55%
==========================================
Files 7 7
Lines 181 181
==========================================
+ Hits 59 60 +1
+ Misses 122 121 -1 ☔ View full report in Codecov by Sentry. |
Everything is just moving to v1.10 because it's the new LTS. And there's also an entire web of changes to have to handle. NonlinearSolve.jl and BoundaryValueDiffEq.jl already required v1.9, LinearSolve.jl had some extensive versioning support that is much easier to maintain if dropped, SciMLSensitivity.jl had tests failing on v1.10 that needed some seemingly v1.10-specific changes, RecursiveArrayTools.jl had some at least v1.9-specific behavior related to This update took about 2 months to complete with multiple contributors, so I don't think it's trivial to backport everything here. If someone wants to get everything back to the previous LTS (v1.6) they can do so, but I would require that the entire SciML stack is re-tested and the full set is fixed up. Otherwise it's simply better to leave it fixed at working versions and simply send back patches as required. I don't think specifically treating v1.9 as special because technically NonlinearSolve.jl is able to do v1.9+ is a good strategy either, given that it's not an LTS and v1.10 is already out. So I think we either need to support both the old and the new LTS, i.e. v1.6 and v1.10, or just the new LTS v1.10. But again, actually making v1.6 work across the board with all of the newest versions of the interfaces would be a much larger set of work than I could capably take on, so the plan is to fix them to versions that work unless someone really wants this feature and they are willing to dedicate the time to actually make it work. And when I say actually make it work, I mean all the way down to the sensitivities since I don't want any incorrect gradients just for some vanity v1.9 support. |
Makes sense, I am bumping the versions to 1.10. |
From git history, it seems like the bump to 1.10 was accidental (at least it was associated with a Aqua testing PR)
Can we allow 1.9 if the tests pass? Currently, this is a dep of SciMLBase, so it essentially means a lot of downstream packages will have to drop 1.9 to depend on the latest SciMLBase.