-
Notifications
You must be signed in to change notification settings - Fork 6
Description
This should be considered low priority.
Is your feature request related to a problem? Please describe
modify_reaction "executes" for surface reactions, but it does not actually work. Though in many cases, it does crash with an error.
Note: I know this is not critical because one can just make a new cti file entirely, and also because I have checked and the set_multiplier function does execute correctly.
Describe the desired solution
It would be great if cantera could get modify_reaction to work with sticking and coverage dependent parameters. (Even better if the species of coverage dependence could be changes this way, too).
Describe alternatives you have considered
I have considered simply using the set_multiplier function. While this does work for many cases, it does not fix the issue of coverage dependence, since that is a dynamic quantity during simulatio
Additional context
I have attached a zipfile with a python file to run to see the issue.
surf_pfr_modify_Reaction_Bug.zip