-
Notifications
You must be signed in to change notification settings - Fork 23
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
Fix the normal mode displacement TS check #768
base: main
Are you sure you want to change the base?
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% 74.35% +0.32%
==========================================
Files 101 103 +2
Lines 28030 28490 +460
Branches 5870 5945 +75
==========================================
+ Hits 20752 21185 +433
- Misses 5803 5826 +23
- Partials 1475 1479 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
3a55c4d
to
06254f9
Compare
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
@@ -9,7 +9,7 @@ | |||
from rmgpy.species import Species | |||
|
|||
import arc.rmgdb as rmgdb | |||
from arc.common import get_logger | |||
from arc.common import generate_resonance_structures, get_logger |
Check failure
Code scanning / CodeQL
Module-level cyclic import Error
arc.common
arc.reaction
definition
import
@@ -9,7 +9,7 @@ | |||
from rmgpy.species import Species | |||
|
|||
import arc.rmgdb as rmgdb | |||
from arc.common import get_logger | |||
from arc.common import generate_resonance_structures, get_logger |
Check failure
Code scanning / CodeQL
Module-level cyclic import Error
arc.common
arc.reaction
definition
import
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.out And when looking at
I think it is also important to point out that ARC is running
|
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.