-
Notifications
You must be signed in to change notification settings - Fork 230
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
Enhanced error message of Arkane GaussianLog loadGeometry #1562
Conversation
d07b87e
to
46551c1
Compare
This looks good to me can you rebase? |
46551c1
to
2f13133
Compare
Codecov Report
@@ Coverage Diff @@
## master #1562 +/- ##
=======================================
Coverage 41.85% 41.85%
=======================================
Files 165 165
Lines 28007 28007
Branches 5713 5713
=======================================
Hits 11721 11721
Misses 15497 15497
Partials 789 789 Continue to review full report at Codecov.
|
rebased |
ready to be merged |
This minor PR is ready to be merged. Could anyone approve it? |
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.
Since it seems like Matt verbally approved this, and it also looks fine to me, I'll go ahead and merge it.
The error massage of the loadGeometry method in Arkane's GaussianLog class was elaborated. If a species has more than 50 atoms, users should add the
iop(2/9=2000)
keyword to the input file for ask Gaussian to print the input orientation geometry.