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

Save conformer jobs in the restart file #125

Merged
merged 4 commits into from
May 1, 2019
Merged

Save conformer jobs in the restart file #125

merged 4 commits into from
May 1, 2019

Conversation

alongd
Copy link
Member

@alongd alongd commented May 1, 2019

Why not? For large species they take a while to run, let's spare running
them again.
If conformers are loaded from a restart file, ARC won't generate
additional conformers for the species (since it thinks it already has,
using the Scheduler.dont_gen_confs list)
We are still generating conformers, though, if guesses were given as
Species.xyz

alongd added 2 commits May 1, 2019 11:49
Couldn't supress it from logging lots of junk
Why not? For large species they take a while to run, let's spare running
them again.
If conformers are loaded from a restart file, ARC won't generate
additional conformers for the species (since it thinks it already has,
using the Scheduler.dont_gen_confs list)
We are still generating conformers, though, if guesses were given as
Species.xyz
@codecov
Copy link

codecov bot commented May 1, 2019

Codecov Report

Merging #125 into master will decrease coverage by 0.02%.
The diff coverage is 5.26%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #125      +/-   ##
=========================================
- Coverage   40.43%   40.4%   -0.03%     
=========================================
  Files          22      22              
  Lines        5755    5761       +6     
  Branches     1530    1535       +5     
=========================================
+ Hits         2327    2328       +1     
- Misses       3048    3051       +3     
- Partials      380     382       +2
Impacted Files Coverage Δ
arc/settings.py 100% <ø> (ø) ⬆️
arc/species/species.py 58.15% <ø> (ø) ⬆️
arc/scheduler.py 20.22% <5.26%> (-0.02%) ⬇️
arc/reaction.py 41.69% <0%> (ø) ⬆️

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 5b39b88...e3216a4. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented May 1, 2019

Codecov Report

Merging #125 into master will decrease coverage by 0.02%.
The diff coverage is 5.26%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #125      +/-   ##
=========================================
- Coverage   40.43%   40.4%   -0.03%     
=========================================
  Files          22      22              
  Lines        5755    5761       +6     
  Branches     1530    1535       +5     
=========================================
+ Hits         2327    2328       +1     
- Misses       3048    3051       +3     
- Partials      380     382       +2
Impacted Files Coverage Δ
arc/settings.py 100% <ø> (ø) ⬆️
arc/species/species.py 58.15% <ø> (ø) ⬆️
arc/scheduler.py 20.22% <5.26%> (-0.02%) ⬇️
arc/reaction.py 41.69% <0%> (ø) ⬆️

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 5b39b88...e3216a4. Read the comment docs.

@alongd alongd merged commit 6738253 into master May 1, 2019
@alongd alongd deleted the restart branch May 1, 2019 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant