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

Enhanced error message of Arkane GaussianLog loadGeometry #1562

Merged
merged 1 commit into from
Mar 27, 2019

Conversation

alongd
Copy link
Member

@alongd alongd commented Mar 15, 2019

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.

@alongd alongd self-assigned this Mar 15, 2019
@alongd alongd added the Arkane label Mar 15, 2019
@alongd alongd changed the title In Gaussian LoadGeometry search for standard orientation Enhanced error message of Arkane GaussianLog loadGeometry Mar 17, 2019
@alongd alongd marked this pull request as ready for review March 17, 2019 00:56
@alongd alongd requested a review from mjohnson541 March 17, 2019 00:56
@mjohnson541
Copy link
Contributor

This looks good to me can you rebase?

@codecov
Copy link

codecov bot commented Mar 22, 2019

Codecov Report

Merging #1562 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 83ad0f2...2f13133. Read the comment docs.

@alongd
Copy link
Member Author

alongd commented Mar 22, 2019

rebased

@alongd
Copy link
Member Author

alongd commented Mar 24, 2019

ready to be merged

@alongd
Copy link
Member Author

alongd commented Mar 27, 2019

This minor PR is ready to be merged. Could anyone approve it?

Copy link
Contributor

@mliu49 mliu49 left a 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.

@mliu49 mliu49 merged commit 425879e into master Mar 27, 2019
@mliu49 mliu49 deleted the gaussian_load_geometry branch March 27, 2019 22:28
@mliu49 mliu49 mentioned this pull request May 15, 2019
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants