isi-vista / vista-pegasus-wrapper

A higher-level API for ISI Pegasus, adapted to the quirks of the ISI Vista group
MIT License
2 stars 1 forks source link

Add Support for jobs with Containers #76

Closed lichtefeld closed 3 years ago

lichtefeld commented 3 years ago

Closes #75 WIP - Currently supports adding containers to the Transformation catalog, next step is supporting adding a container to a job request.

codecov[bot] commented 3 years ago

Codecov Report

Merging #76 (ff97430) into master (32deba8) will increase coverage by 6.14%. The diff coverage is 92.30%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #76      +/-   ##
==========================================
+ Coverage   76.42%   82.57%   +6.14%     
==========================================
  Files          13        9       -4     
  Lines         560      505      -55     
==========================================
- Hits          428      417      -11     
+ Misses        132       88      -44     
Impacted Files Coverage Δ
pegasus_wrapper/__init__.py 56.75% <66.66%> (-0.82%) :arrow_down:
pegasus_wrapper/artifact.py 92.68% <100.00%> (ø)
pegasus_wrapper/conda_job_script.py 66.66% <100.00%> (ø)
pegasus_wrapper/key_value.py 56.47% <100.00%> (ø)
pegasus_wrapper/pegasus_utils.py 100.00% <100.00%> (ø)
pegasus_wrapper/resource_request.py 88.04% <100.00%> (ø)
pegasus_wrapper/workflow.py 96.81% <100.00%> (+0.17%) :arrow_up:
pegasus_wrapper/scripts/sort_nums_in_file.py
pegasus_wrapper/scripts/nuke_checkpoints.py
pegasus_wrapper/scripts/multiply_by_x.py

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 32deba8...ff97430. Read the comment docs.