Skip to content

Add Molpro 2018 unit tests but do not enable them #872

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

Merged
merged 2 commits into from
Aug 3, 2020

Conversation

berquist
Copy link
Member

@berquist berquist commented Jun 9, 2020

Closes #735

The molecular orbital coefficient section has changed quite a bit. This adds the inputs and outputs first, and updating the parser with fixes will come later.

@berquist berquist added the Molpro label Jun 9, 2020
@berquist berquist added this to the v1.6.4 milestone Jun 9, 2020
@langner
Copy link
Member

langner commented Jul 1, 2020

@berquist Is something wrong with this, or is it just not complete? If the latter, perhaps submitting part of what you want is a good step forward?

@berquist berquist changed the title Add Molpro 2018 unit tests Add Molpro 2018 unit tests but do not enable them Jul 3, 2020
@berquist berquist marked this pull request as ready for review July 3, 2020 13:46
@berquist
Copy link
Member Author

berquist commented Jul 3, 2020

Done.

@shivupa
Copy link
Member

shivupa commented Aug 3, 2020

I am going to merge this

@shivupa shivupa merged commit eacd0dc into cclib:master Aug 3, 2020
@berquist berquist deleted the 735-molpro-2018 branch August 5, 2020 01:28
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.

Update unit tests for Molpro 2018.2
3 participants