-
-
Notifications
You must be signed in to change notification settings - Fork 79
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
Reaction expression parser #330
Comments
If you want to put in a PR that makes the parsing code more modular that would be great and appreciated! It would need some work to generate You could always also just build a string with the macro wrapping the reactions, and then |
Could I submit a PR for it? I did something that would fit. |
Certainly! Making it more modular would be great. We've also recently discussed adding a rx = @reaction k, A --> B type macro, so having a standalone parser for such expressions would be good. The plan was to make it default to having |
We support serialization now, so I think that should cover this. |
It'd be nice to isolate the reaction expression parser for use in, e.g., loading reactions from a file. This is simple to do: One can overload
get_reactions
as inget_reactions(exs::Vector{Expr}, reactions = Vector{ReactionStruct}(undef,0))
(where theexs
are the reaction expressions as supplied in@reaction_network
.).I'd be grateful to know if there's a more generic / elegant approach / you've some ideas on this that I'd implement and push.
The text was updated successfully, but these errors were encountered: