-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
working on spatially varying reaction rates, WIP #219
working on spatially varying reaction rates, WIP #219
Conversation
@@ -52,13 +52,20 @@ end | |||
|
|||
update rates of all reactions in rxs at site | |||
""" | |||
function update_rx_rates!(rx_rates, rxs, u, site) | |||
function update_rx_rates!(rx_rates::RxRates{F,M}, rxs, u, site) where {F, M <: MassActionJump} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we still need this version? Shouldn't we always just SpatialMassActionJump
s internally for the spatial solvers now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MassActionJump
has more utilities like parameter mapping and merging so I thought it's good to leave it as a sort of backward compatibility
Want to be able to handle spatially varying reaction rates.