Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Did we use gamma correctly? It's an array, but we only ever use the first element #604

Closed
Tracked by #211
mattldawson opened this issue Jul 15, 2024 · 3 comments
Closed
Tracked by #211
Assignees

Comments

@mattldawson
Copy link
Collaborator

mattldawson commented Jul 15, 2024

If gamma is the same for each stage, only one LU decomposition is used for each stage (page 103 Hairer and Wanner, 1996). This may be why there is only one gamma used. Verify that this really is correct

Ideas

[1] Hairer, E., Wanner, G., 1996. Solving Ordinary Differential Equations II: Stiff and Differential-Algebraic Problems, 2nd edition. ed. Springer, Berlin ; New York.

@K20shores K20shores added this to the MICM, TUV-x in CAM-SIMA milestone Jul 17, 2024
@K20shores K20shores self-assigned this Jul 18, 2024
@K20shores
Copy link
Collaborator

It appears that any gamma past index 0 is only used for non-autonomous equations.

https://github.com/KineticPreProcessor/KPP/blob/main/int/rosenbrock.f90#L612-L615

Autonomous equations are those that are independent of time $F=F(y)$ and non-autonomus equations are dependent on time $F=F(t,y)$. In our case, I think we are autonomous equations.

Even though the jacobian and forcing are physically dependent on time, this information is already accounted for in the calculation of the rate constants which are not recalculated during timestep refinement.

@sjsprecious @mattldawson @mwaxmonsky @boulderdaze. I think we can close this issue because of this. What do y'all think?

@sjsprecious
Copy link
Collaborator

@K20shores I think you are right that the chemistry system is an autonomous equation and the usage of gamma parameter is correct here.

@boulderdaze
Copy link
Collaborator

I think your reasoning is well described for me to believe the system solves autonomous equations. I agree with you.

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

No branches or pull requests

4 participants