compdyn / partmc

Particle-resolved stochastic atmospheric aerosol model
http://lagrange.mechse.illinois.edu/partmc/
GNU General Public License v2.0
27 stars 15 forks source link

Adapt rate_updates to multi-cells #131

Open cguzman95 opened 4 years ago

cguzman95 commented 4 years ago

Mentioned briefly in #125, rxn_data class misses some function to update rxn_solver_id and consequently difference cells (actually, it's set by default to zero and not updated, so when we do update_rata we are going each time to cell 0). I half-fixed it in #129 updating the same rate to all cells, but should be fixed completely in the future.

Since it needs some design on the main fortran code, I want to asign this to you, @mattldawson and avoid some possible re-design of my implementation.

mattldawson commented 4 years ago

sounds good - I may have some questions as I start working on this

cguzman95 commented 4 years ago

Ok no problem. To do a suggestment, I think it should be something like when we create the update_rate object, send as an optional parameter the id cell that we want to set. Then, in the corresponding test create multiple rxn_update_data objects, each one with his own type, rate and cell