-
Notifications
You must be signed in to change notification settings - Fork 81
Add 1.12 to CI #2613
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
base: main
Are you sure you want to change the base?
Add 1.12 to CI #2613
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2613 +/- ##
=======================================
Coverage 67.76% 67.76%
=======================================
Files 58 58
Lines 20920 20920
=======================================
Hits 14176 14176
Misses 6744 6744 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Benchmark Results
Benchmark PlotsA plot of the benchmark results has been uploaded as an artifact at https://github.com/EnzymeAD/Enzyme.jl/actions/runs/18011810485/artifacts/4105812934. |
bdd4d2b to
28b9012
Compare
|
Split the fix here out to #2654 |
33cc20b to
8701e13
Compare
|
@gdalle the current DI version in CI here doesn't support 1.12. Can you open an independent PR bumping to a version which supports 1.12? |
|
What do you mean by "it doesn't support Julia 1.12"? |
|
There's no upper bound on Julia compat in DI |
|
It's JET, a dependency of DifferentiationInterfaceTest, which fails to precompile: https://github.com/EnzymeAD/Enzyme.jl/actions/runs/19831329281/job/56817898039?pr=2613#step:7:262 |
|
Oh right. I'm AFK right now but you can just update to the latest version of DI and DIT, that will do it |
No description provided.