Skip to content
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

Use attributes on rate variable instead of get_rate_stoichiometry() #64

Merged
merged 3 commits into from
Aug 11, 2023

Conversation

sjdaines
Copy link
Member

This means the reaction rate information is available when analysing output, without needing to recreate the model.

New attributes on rate variable used by ReactionNetwork and ReactionNetworkVis:

  • rate_processname::String: a process name (eg "photolysis", "reaction", ...)
  • rate_species::Vector{String} names of reactant and product species
  • rate_stoichiometry::Vector{Float64} stoichiometry of reactant and product species

This means the reaction rate information is available when
analysing output, without needing to recreate the model.

New attributes on rate variable used by ReactionNetwork and ReactionNetworkVis:
- `rate_processname::String`: a process name (eg "photolysis", "reaction", ...)
- `rate_species::Vector{String}` names of reactant and product species
- `rate_stoichiometry::Vector{Float64}` stoichiometry of reactant and product species
@sjdaines sjdaines merged commit c143add into main Aug 11, 2023
3 checks passed
@sjdaines sjdaines deleted the rate_attributes branch August 11, 2023 16:33
@sjdaines
Copy link
Member Author

Also includes fix for #58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant