-
-
Notifications
You must be signed in to change notification settings - Fork 346
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
Revised twin flame #356
Revised twin flame #356
Conversation
The second issue is fixed by 6b588c5. Not actually a numerical issue. |
…ks with cantera2.3.0a2
Based on previous comment, I explicitly added reference to select mixture-averaged model. The example now works with GRI 3.0 |
@@ -1,20 +1,59 @@ | |||
# coding: utf-8 | |||
|
|||
""" |
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.
Is there a reason for removing this summary?
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.
Accidentally omitted. Reinserted in e738001
…ed code in comments
# Use mixture-averaged properties | ||
oppFlame.transport_model = 'Mix' | ||
# Uncomment this line to use a Multi-component formulation. Default is mixture-averaged | ||
#oppFlame.transport_model = 'Mix' |
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.
Should this say 'Multi' to match the comment? Or should the comment be changed?
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.
Thanks for the catch. Changed in 833e850
There are two problems with the existing example (premixed_counterflow_twin_flame.py)
In this commit, I fixed the first problem. I checked it against the Smooke and Giovangigli mechanism for methane, for which this file works. I also tried lots of ways to curv, gradient, etc and various compositions, but gave up. Looks like a numerical (convergence) issue, but there is a good chance that the code will work with other mechanisms that do not ship with Cantera
Secondly, I added a function to compute the consumption speed, since this is a popular use for the twin flame solver. The integration is done using np.trapz, overriding the need for scipy.integrate.trapz as was done in the previous commit