-
-
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
ck2yaml converter gives empty values for coverage_dependancies #956
Comments
Can you please provide a full example, e.g. including the complete Chemkin surface, thermo, and gas phase input files? |
I have attached the mechanism in .txt format here. It doesn't allow me to attach *.inp files. |
speth
added a commit
to speth/cantera
that referenced
this issue
Jan 20, 2021
An indentation error meant that coverage dependencies weren't being written in the (simpler) case where we don't have to modify the units for the activation energy. Fixes Cantera#956
4 tasks
speth
added a commit
that referenced
this issue
Jan 21, 2021
An indentation error meant that coverage dependencies weren't being written in the (simpler) case where we don't have to modify the units for the activation energy. Fixes #956
This comment was marked as off-topic.
This comment was marked as off-topic.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem description
When a surface mechanism from Chemkin is converted into YAML format using the
ck2yaml
command, the fieldcoverage_dependancies
for surface reactions is empty. Another issue is that the reaction numbers are also not updated. (Always shows# Reaction -1
).If the same mechanism is used to convert to CTI format using the
ck2cti
command,coverage_dependancies
are updated accurately.To reproduce
Run the command :
python.exe -m cantera.ck2yaml --input=chem.inp --thermo=therm.dat --transport=tran.dat --surface=surf.inp --permissive
Behavior
The surface reactions in the original Chemkin surface file are -
The converted YAML file shows
System information
Attachments
convertedYAML.txt
convertedCTI.txt
surf.txt
chem.txt
therm.txt
The text was updated successfully, but these errors were encountered: