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

Miscellaneous changes #109

Merged
merged 42 commits into from
Apr 3, 2019
Merged

Miscellaneous changes #109

merged 42 commits into from
Apr 3, 2019

Conversation

alongd
Copy link
Member

@alongd alongd commented Mar 28, 2019

  • Added examples to .gitignore
  • Allow users to specify the number of cpu's to use per server
  • Download additional info re failed jobs (out.txt, err.txt, slurm.out)
  • Don't run orbitals jobs by default
  • Make servers in ess_settings a list
  • Allow users to specify a levels_ess dictionary associating levels of theory (or partial phrases of methods or basis sets) with an ESS
  • Added ess_settings to settings.py instead of the API
  • Tests: functions in main
  • Additional minor fixes

@codecov
Copy link

codecov bot commented Mar 29, 2019

Codecov Report

Merging #109 into master will decrease coverage by 0.13%.
The diff coverage is 28.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #109      +/-   ##
==========================================
- Coverage   40.55%   40.41%   -0.14%     
==========================================
  Files          22       22              
  Lines        5265     5379     +114     
  Branches     1373     1402      +29     
==========================================
+ Hits         2135     2174      +39     
- Misses       2782     2854      +72     
- Partials      348      351       +3
Impacted Files Coverage Δ
arc/job/submit.py 100% <ø> (ø) ⬆️
arc/job/inputs.py 100% <ø> (ø) ⬆️
arc/job/ssh.py 12.56% <0%> (-0.41%) ⬇️
arc/arc_exceptions.py 100% <100%> (ø) ⬆️
arc/settings.py 100% <100%> (ø) ⬆️
arc/scheduler.py 19.27% <11.42%> (-0.07%) ⬇️
arc/species/species.py 58.7% <39.13%> (-0.26%) ⬇️
arc/main.py 44.13% <39.5%> (+1.93%) ⬆️
arc/species/converter.py 71.3% <50%> (-0.49%) ⬇️
arc/job/job.py 18.67% <6.41%> (-1.69%) ⬇️
... and 3 more

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 e4c85a6...ea19450. Read the comment docs.

@alongd
Copy link
Member Author

alongd commented Mar 29, 2019

Rebasing a personal branch with commits from this PR will probably cause conflicts (in settings.py and submit.py) - it's recommended to make backups of your modifications, and manually add them after updating ARC

alongd added 12 commits April 2, 2019 21:35
Also removed settings.py which has no effect, since git cannot ignore
files it started tracking
Download out.txt and err.txt from OGE
and slurm_<job id>.out from Slurm
to download and get modified time
since this directive doesn't actually visuallizes anything
Also changed the default to False, since this might take too long
(longer than all other jobs for the species)
This could be read by RMG as the species index
use `_` instead
and removed precedence from servers dictionary

Also modified determine_ess_settings() accordingly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant