Torkel Loman

Since its recent update, ModelingToolkit requires the system independent variable to be given to `ODESystem`s. I.e. you have to add `t` after adding the equations: ```julia # define the model...

Actually, just moved everything to the latest Catalyst and MTK version, and now things seem to work. Will still need to go through things myself to dice if it is...

I think we discussed this generally as well? https://github.com/SciML/Catalyst.jl/issues/88 I feel there is quite couple of theoretical considerations, so happy wo use whichever approach you think is appropriate. Not sure...

Either the last two lines should be removed, or there is something here which is not working as intended.

Have you tried fitting parameters using PEtab (https://github.com/sebapersson/PEtab.jl?tab=readme-ov-file)? Generally, it is more robust then using DiffEqParamEstim. I'd probably recommend that as the go-to choice, and think it should work for...

Yeah, the PEtab tutorial was dropped while PEtab was updating to the latest MTK version (which was recently completed, so I will try to reenable that one). There is decent...

I'm not keen on doing this right now. Maybe later when I can look at it properly. Last few days have been quite hectic, so haven't been able to look...

It seems potentially a lot more overhead in that we have to manage several additional packages. And we'd want to have those as downstream tests anyway, so I'm not sure...

Moving them out still would not change whether they are broken or not? Short term the whether the functionality does/does not work won't really change. Long term an argument can...

I will try help figure out what is going on as well. The MTK Bifurcation tests do seem to work, so its not fundamentally broken. Ours are more extensive though