-
Notifications
You must be signed in to change notification settings - Fork 71
Change in behavior for reinterpreted arrays between Julia 1.10 and Julia 1.11 #2382
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
Comments
This comment has been minimized.
This comment has been minimized.
This one is related to the Core 2 SciMLSensitivity tests, https://github.com/SciML/SciMLSensitivity.jl/actions/runs/14969197208/job/42046022185 |
We one is deprioritized a bit from the others since we found a viable workaround SciML/SciMLSensitivity.jl#1191 |
this one is a user-side error, you're marking a non-differentiable type as differentiable |
Sorry, probably shouldn't have conflated those, I just wasn't sure how it was actually different from the other example and I was trying to find a simpler working example. The actual error is the |
ok cool, well in any case if you can make a more minimal example that has the same error (equally applies for all the other issues), it'll allow us to get these fixed faster |
Yeah for sure, the example at the very top is the result of many hours of figuring out exactly how SciMLSensitivity calls Enzyme with what types and trying to recreate it. For some reason I can't get the exact same error message without using |
perhaps cc @oscardssmith for support |
this is probably Enzyme strugling with |
No SciMLSensitivity unwraps. |
Yeah they get unwrapped, none of the functions in the MWE are FunctionWrappers or FunctionWrapperWrappers. |
Uh oh!
There was an error while loading. Please reload this page.
I'm getting an issue where the behavior around reinterpreted arrays is different between Julia 1.10 and Julia 1.11.
Julia 1.11:
gives
If you do this (note the upper case
Nothing
s):it works fine. The only difference is that
ytmp
etc. are no longerReinterpretedArrays
. The thing is, the first one works on Julia 1.10:You can run this without problems,
One thing is that this:
works perfectly fine on Julia 1.11, even though they should essentially be doing the exact same thing I think.
Another thing is that if you specify
SciMLBase.NoSpecialize
that gets rid of the error as well:The text was updated successfully, but these errors were encountered: