Skip to content
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

Enhancing heteroatoms in RMG-database #194

Closed
wants to merge 61 commits into from
Closed

Enhancing heteroatoms in RMG-database #194

wants to merge 61 commits into from

Conversation

alongd
Copy link
Member

@alongd alongd commented Jun 3, 2017

This PR contains sulfur and some nitrogen enhancements:

  • Added new kinetic & thermo libraries: SOxNOx, Sulfur/GlarborgBozzeli,, Sulfur/GlarborgMarshall, Sulfur/NS, Sulfur/H2S, BurcatNS
  • Added training reactions (Disprop, R_Add_MulBond, R_Rec, H_Abs)
  • Sulfur radical corrections and GAV (thanks @rgillis8!)
  • Added sulfur groups to nearly all families in RMG
  • Added two new families (2+2_cycloaddition_CS, Cyclic_Thioether_Formation)

This should be merged together with the corresponding RMG-Py PR.

@mention-bot
Copy link

@alongd, thanks for your PR! By analyzing the history of the files in this pull request, we identified @jwallen, @connie and @nyee to be potential reviewers.

alongd and others added 25 commits July 5, 2017 11:51
Thermo format for HOSOH was changed to account for temperatures higher than 1500 K
Non-sulfur reactions were removed (except for O3 reactions, which currently RMG lacks)
Commented-out training reaction #8 which now has a better rate estimate
(at combustion-relevant temperatures, the williams et al. study is for low T)
Removed training reaction #9 which is incorrect: NO2-2 (O atom has the
rad in this resonance structure) reacts with OH to form HOONO,
not HNO3 as stated in this reaction; this is also not in agreement
with the cited paper by Williams et al.
alongd added 24 commits July 5, 2017 11:51
Added heteroatom groups to Diels_alder_addition Family
At this stage only top tree nodes (L1's) were elaborated.
No sub groups for heteroatoms were created due to the enormouse number of combinations.
If a need for better resolution groups raises for this family, we'll add them.
@alongd
Copy link
Member Author

alongd commented Oct 13, 2017

This PR is outdated.
Relevant commits from this PR are included in #215 which will soon be merged.

@alongd alongd closed this Oct 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants