-
-
Notifications
You must be signed in to change notification settings - Fork 79
Tests failing for prerelease #1235
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
I wouldn't worry about it yet. I think it's a bug in the prerelease that is breaking some world age behavior due to some changes in globals. I'll need to talk with @oscardssmith about that. |
Ok, thanks for letting us know. |
yeah it is a change in globals. 1.12 got a tiny bit stricter in ways that pretty much only affect code that was slightly weird/broken which given the amount of code we have, it isn't exactly surprising that we have some of it. |
Some of it could be things we need to change, but other things may just be accidental breaking changes in the Julia compiler that we need to notify get fixed during the beta stage 😅 . You can be the arbiter of which is which. Let's walk through that in one of the Monday calls since I assume some of these might need discussion (I see RuntimeGeneratedFunctions popping up) |
Let's just keep this to the issue SciML/RuntimeGeneratedFunctions.jl#95 (comment) |
This should be at least a partial fix to #1235, but may also need SciML/RuntimeGeneratedFunctions.jl#96 to fully get pre working
It looks like we know what the RGF one is, but the Catalyst one looks like it's just incorrect macro code. Catalyst.jl/src/reactionsystem.jl Lines 1187 to 1188 in 005799c
You should never use eval in a macro. The rule is that if you use eval in a macro is probably wrong 😅. |
That was an internal hack to allow unpacking all the Symbolics in a model, and was never exported or advertised/made api. It really isn’t needed anymore now that |
That would be best, because it seems like it's just not going to be easy to support that in Julia v1.12 as it's doing some odd writing to modules at eval time. Working around it seems like a bit more work and is likely not very reliable. |
We need to investigate this and see if the issue is Catalyst related or not.
The text was updated successfully, but these errors were encountered: