Sam Isaacson

Results 700 comments of Sam Isaacson

I think for bio readers this is a nice one that is focused on algorithms: Luca Marchetti, Corrado Priami, Vo Hong Thanh, "Simulation Algorithms for Computational Systems Biology".

We should probably benchmark that case and see if there is a speed up. I don’t think we’ve ever done so.

Can you change the base branch for all these PRs you are opening to v14 so they are readable?

`ReactionSystem`s are `AbstractSystem`s that conform to the MTK interface, shouldn't SII work with a general `AbstractSystem`?

Isn’t the default usage of `connect(sys1.a, sys2.b)` that they are equal (unless there is metadata that they are flow variables or such)? I was thinking we just allow these as...

Then just merge when this is done, irrespective of the tests failing.

@TorkelE ended up adding this as part of Catalyst 14, but thanks for your getting this started!

Alternatively one can format using v1.0.32 of JuliaFormatter, which CI is now fixed to use. That should avoid reformatting the entire repo.

We have [conservationlaw_constants](https://docs.sciml.ai/Catalyst/stable/api/catalyst_api/#Catalyst.conservationlaw_constants) to get them.

That said, attaching metadata to them is probably a good approach to propagate their meaning through to downstream system types and allow users to extract them from such systems.