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

AEC BAC for ωB97X-D3/def2-TZVP and B97-D3/def2-mSVP #459

Merged
merged 8 commits into from
Mar 22, 2021

Conversation

kspieks
Copy link
Contributor

@kspieks kspieks commented Mar 14, 2021

This PR adds frequency scaling factors, AEC, and BAC for two levels of theory: ωB97X-D3/def2-TZVP and B97-D3/def2-mSVP. All calculations were run with QChem 5.3

Frequency scaling factors were computed using ARC.

AECs were determined by running sp calculations of the 16 small molecules from the reference database. The experimental geometries were used i.e. no optimization was performed.

BACs were determined by running opt + freq jobs in QChem using ARC. The reference database previously had stored the optimized geometries from ωB97M-V/def2-TZVPD; these geometries were used as the initial guess. Rotors were not considered. A few molecules failed during QM by either not converging during geometry optimization or yielding a negative frequency despite being a stable species, not a TS; these molecules were omitted and the remaining 400+ molecules were used for BACs fitting.

For ωB97X-D3/def2-TZVP:

  • RMSE/MAE before fitting: 6.81/5.13 kcal/mol
  • RMSE/MAE after fitting: 2.21/1.33 kcal/mol

1_wb97xd3_def2tzvp_errors.pdf

For B97-D3/def2-mSVP:

  • RMSE/MAE before fitting: 27.85/21.78 kcal/mol
  • RMSE/MAE after fitting: 6.58/3.16 kcal/mol

1_b97d3_def2msvp_errors.pdf

@kspieks kspieks self-assigned this Mar 14, 2021
@alongd
Copy link
Member

alongd commented Mar 15, 2021

@kspieks, could you make a twin RMG-Py PR to update the model chemistry table in the docs?

@kspieks
Copy link
Contributor Author

kspieks commented Mar 15, 2021

@alongd Will do! Thanks for reminding me

kspieks added 7 commits March 16, 2021 15:48
Note that 6 species failed during QM due to the frequency being negative for spcs112, spcs229, spcs303, spcs351, spcs407, spcs419

Note that 4 species failed to converge during QM optimization: spcs149, spcs161, spcs170, spcs397  

I tried changing convergence criteria, but could not fix the issue. The remaining 411 species were used when fitting BACs.
Note that 17 species failed during QM due to the frequency being negative  or optimization failing to converge: spcs68, spcs117, spcs126, spcs140, spcs177, spcs229, spcs223, spcs286, spcs339, spcs351, spcs149, spcs161, spcs170, spcs397, spcs195, spcs267, and spcs320 

The remaining 404 species were used for fitting BACs.
@kspieks kspieks force-pushed the aec_bac_wb97xd3_b97d3 branch from 4ffebb3 to 772cb4e Compare March 16, 2021 19:50
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.

Thanks, Kevin! This is well documented. I guess it is acceptable to have those parameters fitted without rotor and non-converged species.

@kspieks
Copy link
Contributor Author

kspieks commented Mar 21, 2021

Thanks, Kevin! This is well documented. I guess it is acceptable to have those parameters fitted without rotor and non-converged species.

@xiaoruiDong Does this PR look ready to be merged?

I’m unsure why the continuous integration of rmg-tests are failing, but it seems like that has failed for many recent PRs. I can add it as a hackathon topic for us to look into.

Regarding conformers, my understanding was that the experimental geometry from CCCBDB was used as the initial guess for ωb97mv optimization (whose converged geometry was the initial guess for my calculations). It’s possible Colin did a conformer search with ARC for these species, but I’m unsure where the log files are so I can’t check for sure. Looks like other PRs (#412) for adding BACs also did not consider rotors (or the PR (#417) doesn't comment on it).

@xiaoruiDong
Copy link
Contributor

@kspieks Thanks. This PR looks good. Can you also update the reference database yamls with D97-B3/def2-mSVP energies? Once they are updated, please rebase the PR, and I will merge it into the master branch. Thanks!

@kspieks
Copy link
Contributor Author

kspieks commented Mar 22, 2021

@kspieks Thanks. This PR looks good. Can you also update the reference database yamls with D97-B3/def2-mSVP energies? Once they are updated, please rebase the PR, and I will merge it into the master branch. Thanks!

Thanks for reviewing! I updated the reference database with b97d3/def2-msvp data :)

@kspieks
Copy link
Contributor Author

kspieks commented Mar 10, 2023

FYI: all log files and Arkane outputs related to these jobs have been copied to my account on our new greencloud sever at /volume1/home/users/kspieker/AEC_BAC_calculations

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.

3 participants