WIP-Adding software version retrieval

[rcali21]

No description provided.

[codecov]

Codecov Report

Base: 77.13% // Head: 76.77% // Decreases project coverage by -0.35% ⚠️

Coverage data is based on head (4b9df08) compared to base (67f7e72).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #580      +/-   ##
==========================================
- Coverage   77.13%   76.77%   -0.36%     
==========================================
  Files          20       19       -1     
  Lines        4330     4349      +19     
  Branches     1217     1193      -24     
==========================================
- Hits         3340     3339       -1     
- Misses        802      821      +19     
- Partials      188      189       +1     

Flag	Coverage Δ
unittests	76.68% <100.00%> (-0.36%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
pydra/engine/audit.py	91.66% <100.00%> (+1.34%)	⬆️
pydra/engine/submitter.py	77.84% <0.00%> (-10.48%)	⬇️
pydra/engine/specs.py	88.00% <0.00%> (-0.59%)	⬇️
pydra/__init__.py
pydra/engine/core.py	89.28% <0.00%> (+0.20%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[djarecka]

@rcali21 - have you seen the errors from the tests. Please let me know if something is not clear from the error messages.

Also, it would be good to expand the tests to check this specific change (after you fix the current tests)

[rcali21]

Yes, I will be working on resolving them and building a test, thank you.

[rcali21]

@djarecka Please weigh in on this when you see it, thank you.

[rcali21]

Would the singularity and SLURM calls be failing because I implemented something that uses stdout?

[djarecka]

don't worry about Singularity right now, I think it might be unrelated. Will try to check it locally

[djarecka]

@rcali21 - I fixed the issue with SIngularity, you could merge the current master to your branch and the tests should pass

[djarecka]

thanks for adding a test, but perhaps you can also add version check to the previous test: test_audit_task