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

More ketoenol rxns #571

Merged
merged 4 commits into from
Mar 30, 2022
Merged

More ketoenol rxns #571

merged 4 commits into from
Mar 30, 2022

Conversation

kspieks
Copy link
Contributor

@kspieks kspieks commented Mar 23, 2022

This PR adds 71 training reactions to the ketoenol family and refits the rate tree. Originally, there were only 6 training reactions. After this PR (as well #565 and #568), there are now just over 100 training reactions so our ketoenol rate estimates have gotten substantially better. All species were calculated at CCSD(T)-F12a/cc-pVDZ-F12//ωB97X-D3/def2-TZVP. A thorough conformer search was done with ACS on the reactant, TS, and product. 1D rotor scans were also done on the reactant, TS, and product. All scans start and end in the same place and have the lowest energy well at 0 degrees, which makes sense since a conformer search was done for each species. The notebook below has assert glog.success which was True for all species. Also included in the zip file is the notebook for running ATG to refit the rate rules.

20220323_ketoenol.zip

@kspieks kspieks requested a review from xiaoruiDong March 23, 2022 21:19
@kspieks kspieks self-assigned this Mar 23, 2022
@kspieks kspieks mentioned this pull request Mar 25, 2022
@kspieks kspieks force-pushed the more_ketoenol_rxns branch from d880dc5 to 9695c97 Compare March 28, 2022 22:35
@kspieks
Copy link
Contributor Author

kspieks commented Mar 28, 2022

@xiaoruiDong Thanks so much for your help in reviewing this PR today. I incorporated the changes we discussed. The notebook below is updated with the new fitting.
7_tree_fitting_notebook_ketoenol.ipynb.zip

@kspieks kspieks mentioned this pull request Mar 28, 2022
Copy link
Contributor

@xiaoruiDong xiaoruiDong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kspieks Thanks for correcting the PR. It looks good to me.

@xiaoruiDong xiaoruiDong merged commit 3c823db into main Mar 30, 2022
@xiaoruiDong xiaoruiDong deleted the more_ketoenol_rxns branch March 30, 2022 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants