-
Notifications
You must be signed in to change notification settings - Fork 228
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
Added the C2H4 +O surface as a kinetic library #1216
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1216 +/- ##
=======================================
Coverage 43.75% 43.75%
=======================================
Files 171 171
Lines 27935 27935
Branches 5454 5454
=======================================
Hits 12223 12223
Misses 14886 14886
Partials 826 826 Continue to review full report at Codecov.
|
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.
I made one comment, but otherwise it looks good. The documentation built from this was formatted properly. Let me know what you think about the caption, and then this should be ready to merge.
@@ -71,6 +71,8 @@ Below is a list of pre-packaged kinetics library reactions in RMG: | |||
+---------------------------------------+------------------------------------------------------------------------------------------+ | |||
|BurkeH2O2inN2 |Comprehensive H2/O2 kinetic model in N2 atmosphere | | |||
+---------------------------------------+------------------------------------------------------------------------------------------+ | |||
|C2H4+O_Klipp2017 |C2H4 + O (including intersystem crossing), probably important for all C/H/O combustion | |
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.
The library is just the intersystem crossing reactions right? If so the description should be the following:
C2H4 + O intersystem crossing reactions, probably important for all C/H/O
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.
Done
@mliu49, can you merge this pull request for me? |
Added the C2H4+O_Klipp2017 library to documentation.
This is a twin PR of ReactionMechanismGenerator/RMG-database#219