Conversation
eda345f to
bdbee4f
Compare
a9f8714 to
1337c95
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #768 +/- ##
==========================================
+ Coverage 74.03% 75.28% +1.24%
==========================================
Files 101 103 +2
Lines 28030 29480 +1450
Branches 5870 6322 +452
==========================================
+ Hits 20752 22194 +1442
- Misses 5803 5812 +9
+ Partials 1475 1474 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
3a55c4d to
06254f9
Compare
|
Currently it appears the branch has an issue when it comes to parsing the energy of a fine opt job since it attempts to parse the fine opt before it is completed Here is an input file that I have seen the issue occur project: "rmg_rxn_1216"
level_of_theory: "wb97xd/def2tzvp"
job_types:
rotors: False
irc: False
fine: True
sp: True
compute_thermo: False
allow_nonisomorphic_2d: False
species:
- label: "rmg_rxn_1216_r1_ON=O"
smiles: "ON=O"
- label: "rmg_rxn_1216_r2_sbr-C-sbr#N"
smiles: "[C]#N"
- label: "rmg_rxn_1216_p1_sbr-O--sbrsbr-N+-sbr=O"
smiles: "[O-][N+]=O"
- label: "rmg_rxn_1216_p2_C#N"
smiles: "C#N"
reactions:
- label: "rmg_rxn_1216_r1_ON=O + rmg_rxn_1216_r2_sbr-C-sbr#N <=> rmg_rxn_1216_p1_sbr-O--sbrsbr-N+-sbr=O + rmg_rxn_1216_p2_C#N"The exception: Exception has occurred: InputError (note: full exception trace is shown but execution is paused at: _run_module_as_main)
Could not find file /home/calvin.p/runs/ARC/rmg_rxn_1216/calcs/Species/rmg_rxn_1216_p1_sbr-O--sbrsbr-Np-sbr=O/opt_a2957/output.out
File "/home/calvin.p/Code/ARC/arc/parser.py", line 325, in parse_e_elect
raise InputError(f'Could not find file {path}')
File "/home/calvin.p/Code/ARC/arc/scheduler.py", line 2733, in post_sp_actions
zpe_scale_factor=zpe_scale_factor)
File "/home/calvin.p/Code/ARC/arc/scheduler.py", line 1306, in run_sp_job
level=level,
File "/home/calvin.p/Code/ARC/arc/scheduler.py", line 1544, in spawn_post_opt_jobs
self.run_sp_job(label)
File "/home/calvin.p/Code/ARC/arc/scheduler.py", line 622, in schedule_jobs
self.spawn_post_opt_jobs(label=label, job_name=job_name)
File "/home/calvin.p/Code/ARC/arc/scheduler.py", line 519, in __init__
self.schedule_jobs()
File "/home/calvin.p/Code/ARC/arc/main.py", line 646, in execute
output=self.output,
File "/home/calvin.p/Code/ARC/ARC.py", line 65, in main
arc_object.execute()
File "/home/calvin.p/Code/ARC/ARC.py", line 69, in <module>
main()
File "/home/calvin.p/mambaforge/envs/arc_env/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/calvin.p/mambaforge/envs/arc_env/lib/python3.7/runpy.py", line 193, in _run_module_as_main (Current frame)
"__main__", mod_spec)
arc.exceptions.InputError: Could not find file /home/calvin.p/runs/ARC/rmg_rxn_1216/calcs/Species/rmg_rxn_1216_p1_sbr-O--sbrsbr-Np-sbr=O/opt_a2957/output.outAnd when looking at I think it is also important to point out that ARC is running |
and identify Gaussian log files
Added r_bonds_only to Reaction.get_bonds()
deleted a redundant test and minor modifications
Use the correct key
To use if perception of b_mol is problematic Also added number_of_radicals to converter molecules_from_xyz()
and a minor efficiency improvement
This is a large PR that fundamentally adds the Molecule class internally in ARC. Now ARC is compatible with Py 3.12 This PR follows the work done in #754 where ARC were given capabilities to work directly with the RMG-database without using the RMG API. Now, finally, ARC does not depend on Julia. The major change made in this PR required additional modifications. Although we appreciate small and smart PR's it is very hard to decouple these changes. The main modifications are as follows: 1. Naturally, the interface to Arkane had to be modified since now RMG is not a direct dependency. We now use Arkane only as a subprocess for statmech. 2. ESS paersing was previously somewhat facilitated through Arkane, now there's a new `parser` module in ARC with adapters for each ESS. 3. Bugs were discovered in our two perception algorithms (we used to have `species/xyz_to_2d` and `species/xyz_to_smiles`, falling back to a single bond version of the molecule if it cannot be perceived). Now we have a new `species/perceive` algorithm, with a fallback to `species/xyz_to_smiles` if needed. Success rates are higher, and we always return a molecule with bond orders. This might be the end of the `allow_nonisomorphic_2d` flag in ARC. We'll keep it around for a while, but may deprecate it in the future. 4. QCElemental was removed as a dependency. Combined with the removal of Arkane's API, this means that now we provide translations from atomic numbers to atomic mass. this is done in common.py with data stored under `data/elements.yml`. 5. The removal of QCElemental also impacted our atom mapping algorithm. It turns our that we used this as a fall back quite often for isomerization. The atom mapping engine and driver were updated, which is another positive outcome of this PR. 6. Better and automated installation scripts were added under `devtools` for all the external dependencies, and the CI was updated as well, along with the Makefile. A big thanks to @calvinp0 for the endless hours he invested in this. 7. ZMatrices and the H Abstraction heuristics modules were updated as well. 8. The TS NMD checks have been updated, incorporating #768 into this PR. Tests were of course added. We still need to updated the docs, specifically for the installation instructions, and check the installation scripts again, they were mainly tested in the context of the CI. With this merged, we should soon tag a new version of ARC.
|
implemented by #789 |
The normal model displacement check was fixed, a new module for this check was added.
Tests added, though more thorough testing for the overall functionality are still needed.