-
Notifications
You must be signed in to change notification settings - Fork 169
Gaussian v2 parser #1595
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
base: main
Are you sure you want to change the base?
Gaussian v2 parser #1595
Conversation
for more information, see https://pre-commit.ci
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1595 +/- ##
==========================================
+ Coverage 12.07% 12.67% +0.59%
==========================================
Files 110 111 +1
Lines 16818 17095 +277
==========================================
+ Hits 2031 2167 +136
- Misses 14787 14928 +141 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
if line[5:33] == "Natural Orbital Coefficients": | ||
parsed_atombasis = natural_orbital_single_spin_parsing(file_handler) | ||
return {atombasis.__name__: parsed_atombasis} | ||
|
||
if line[5:39] == "Alpha Natural Orbital Coefficients": | ||
parsed_atombasis = natural_orbital_single_spin_parsing(file_handler) | ||
return {atombasis.__name__: parsed_atombasis} | ||
if line[5:38] == "Beta Natural Orbital Coefficients": | ||
parsed_atombasis = natural_orbital_single_spin_parsing(file_handler) | ||
return {atombasis.__name__: parsed_atombasis} |
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 would combine these three into a single conditional.
this_dispersionenergies.append(dispersion) | ||
return {dispersionenergies.__name__: np.array(this_dispersionenergies)} | ||
return None | ||
# The geometry convergence targets and values are printed in a table, with the legends |
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.
Stray comment from Psi4 section
This implements all of the currently implemented V2 attributes except
mocoeffs
andatomcharges
for Gaussian