Sam Isaacson

Results 506 comments of Sam Isaacson

Yeah, somehow we never got this more at the fore-front as a feature even though it is in the docs in several places: https://docs.sciml.ai/Catalyst/stable/catalyst_applications/homotopy_continuation/#Systems-with-conservation-laws https://docs.sciml.ai/Catalyst/stable/faqs/#How-to-index-solution-objects-using-symbolic-variables-and-observables?

@TorkelE what do you think about defaulting it to true? I know we've discussed but I can't remember if you had a reason not to do it by default for...

We can probably modify the function to allow one to provide a keyword to override this error and return the symbolic matrix, but note that no higher-level Catalyst functionality that...

We can add this in eventually, but right now we are hung up from making new releases due to waiting for ModelingToolkit V9 (and then needing to update Catalyst to...

> Change which order with which variables/vertexes are stored (so new order is first all variables at site 1, then all variables at site 2, and so on). Have you...

Adding this macro seems like a good idea, and less clunkier than the current approach. So I'd be in favor of that. However, at a high level I don't like...

Yeah, why don't you get the functionality in here without changing the symbol mappings. Then we can discuss ways to improve their construction as a separate issue / PR.

Also, and I haven't looked at your code closely, but can you make sure to still support the old style noise scaling for now, but add a deprecation warning that...

OK, but if someone passes it the previous way via a kwarg it should now give a deprecation warning but still work, unless they pass it both the new way...