ReproNim / reproman

ReproMan (AKA NICEMAN, AKA ReproNim TRD3)
https://reproman.readthedocs.io
Other
24 stars 14 forks source link

ENH: small tune ups to "jobs -s" output in case of missing external (datalad) #516

Open yarikoptic opened 4 years ago

yarikoptic commented 4 years ago

See individual commits. Now (original version) would show

$> reproman --dbg jobs -s
[status: N/A - needs datalad] 20200107-213600-f814 on smaug via condor$ '{inputs}' '{outputs}' participant --participan...
...

instead of immediately crashing with

$> reproman  jobs -s    
2020-05-25 20:18:57,108 [ERROR  ] DataLad is required for orchestrator 'datalad-pair-run' is missing. [orchestrators.py:__init__:488] (MissingExternalDependency) 

edit1: pushed fb4e2a680a37c1fe660948263d554df1019ea109 which would just catch all exceptions and issue warning while providing NA for the status

codecov[bot] commented 4 years ago

Codecov Report

Merging #516 into master will increase coverage by 0.11%. The diff coverage is 77.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #516      +/-   ##
==========================================
+ Coverage   89.47%   89.59%   +0.11%     
==========================================
  Files         148      148              
  Lines       12290    12295       +5     
==========================================
+ Hits        10997    11016      +19     
+ Misses       1293     1279      -14     
Impacted Files Coverage Δ
reproman/support/jobs/orchestrators.py 91.84% <ø> (ø)
reproman/interface/jobs.py 97.14% <77.77%> (-1.86%) :arrow_down:
...eproman/interface/tests/test_backend_parameters.py 100.00% <0.00%> (+4.34%) :arrow_up:
reproman/tests/skip.py 97.75% <0.00%> (+4.49%) :arrow_up:
reproman/distributions/tests/test_venv.py 98.07% <0.00%> (+10.57%) :arrow_up:

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 b6528a1...fb4e2a6. Read the comment docs.