8000 WIP-Adding software version retrieval by rcali21 · Pull Request #580 · nipype/pydra · GitHub
[go: up one dir, main page]

Skip to content

WIP-Adding software version retrieval #580

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

Merged
merged 24 commits into from
Oct 28, 2022
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
572bdc3
New branch for adding io tracking in PROV
rcali21 Sep 19, 2022
bdf00af
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Sep 19, 2022
11b9f0a
Initial attempt to include input files by gathering from input_spec
rcali21 Sep 21, 2022
023fd6e
Merge https://github.com/rcali21/pydra into PROV_io
rcali21 Sep 21, 2022
d63da2c
Merge branch 'PROV_io' of https://github.com/rcali21/pydra into PROV_io
rcali21 Sep 21, 2022
896ffff
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Sep 21, 2022
f483b36
First attempt at implementing version query
rcali21 Oct 12, 2022
a297565
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Oct 12, 2022
475ec90
Deleted print statements
rcali21 Oct 12, 2022
608b23f
Deleted print statements
rcali21 Oct 12, 2022
00c9573
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Oct 12, 2022
321c00e
Implemented --version call in PROV as well as tests
Oct 15, 2022
2624942
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Oct 15, 2022
eb7b894
Removed 'mrcat' call from test_task.py and replaced with 'less'
Oct 15, 2022
d546841 8000
Removed 'mrcat' call from test_task.py and replaced with 'less'
Oct 15, 2022
c0d62ff
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Oct 15, 2022
712994b
String formatting and flake8 issues resolved.
Oct 15, 2022
2aff250
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Oct 15, 2022
08def18
String formatting and flake8 issues resolved(2).
Oct 15, 2022
2080652
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Oct 15, 2022
82f340d
Initial attempt at adding infile tracking
Oct 15, 2022
4af7944
Added version check to test_audit_task
Oct 28, 2022
e2eb028
Merged changes
Oct 28, 2022
4b9df08
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Oct 28, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
String formatting and flake8 issues resolved(2).
  • Loading branch information
Ryan Cali authored and Ryan Cali committed Oct 15, 2022
commit 08def18c68d76c74b308bcf02d0fe9cf7510b238
9 changes: 5 additions & 4 deletions pydra/engine/audit.py
Original file line number Diff line number Diff line change
Expand Up @@ -186,10 +186,11 @@ def audit_task(self, task):
if command is not None:
cmd_name = command.split()[0]
software = f"{cmd_name} --version"
# take the first word of command as the name of the executable (this may not always be the case)
version_cmd = sp.run(software, shell=True, stdout=sp.PIPE).stdout.decode(
"utf-8"
)
# take the first word of command as the
# name of the executable
# (this may not always be the case)
version_cmd = sp.run(software, shell=True,
stdout=sp.PIPE).stdout.decode("utf-8")
try:
version_cmd = version_cmd.splitlines()[0]

Expand Down
0