Closed Julian-Ufert closed 7 months ago
⚠️ One or more regression tests failed. Please download the failed results and run the tests locally or check the log to see why.
beep boop this comment was written by a bot :robot:
LGTM! Thanks for the fix. Will get a second review to double check. @sevyharris could you confirm that this does what the linked issue needs?
I'll check it out and let you know
Yes, I can confirm I was able to reproduce the error and then these changes fixed the error. (Sorry I seem to have missed your message a year ago about checking this out again) Looks good to me!
No worries! Thanks for verifying. Please approve and merge at your leisure
⚠️ One or more regression tests failed. Please download the failed results and run the tests locally or check the log to see why.
beep boop this comment was written by a bot :robot:
Motivation or Problem
As pointed out by Sevy in issue #2266, the file RMG-Py/arkane/ess/psi4.py (a parser for Psi4 output files) caused the statement
in the file anaconda3/envs/rmg_env/lib/python3.7/site-packages/cclib/bridge/init.py to be evaluated as true, even though the package psi4 is not part of the rmg environement (at least not the standard one). The subsequent line trying to import makepsi4 caused an error message, because cclib.bridge.cclib2psi4 tries to import from psi4.
Description of Changes
I changed the name of the file psi4.py to the more descriptive psi4_parser.py, and the corresponding lines in the files RMG-Py/arkane/ess/init.py and RMG-Py/test/arkane/ess/psi4Test.py referencing the file psi4.py.
Testing
After the change, RMG-Py/test/arkane/ess/psi4Test.py and RMG-Py/arkane/ess//arkaneGaussianTest.py run without errors.