Christopher Tessum

Results 81 comments of Christopher Tessum

Thanks, everyone, for your responses.

Here is a start: https://github.com/SciML/Catalyst.jl/pull/1320 It is enough to get my use-case working, but the API changes I think need further thought and I haven't even looked at the test.

I don't have a specific case of it failing, I was just looking through the code and it seemed wrong.

Thanks for your response! It is defined here: https://github.com/EarthSciML/GasChem.jl/blob/main/src/SuperFast.jl#L33 Let me know if I misunderstood your comment, or if additional information would be useful.

It should run out of the box if you install all the packages at the top. Let me know if it doesn't

Hello again! As a follow-up to the above, I notice that https://github.com/SciML/MethodOfLines.jl/commit/186809073a576797944dad92e30cdf93450af699 deletes the extension with the explanation "move extension over". Where has the extension been moved to? Should it...

MTK [version 9.34](https://github.com/SciML/ModelingToolkit.jl/releases/tag/v9.34.0) supports non-SI units as described here: https://github.com/SciML/ModelingToolkit.jl/pull/2621 . However Catalyst is currently pinned to MTK v9.32. So it may just be as simple as updating that dependency.

> Catalyst isn't pinned to 9.32; are you sure this isn't something you did locally? I currently have ModelingToolkit 9.35.1 with Catalyst 14.4 locally. > > Note that the line...

This may be related to https://github.com/SciML/ModelingToolkit.jl/issues/3017