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

Task Generation #2

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Task Generation #2

wants to merge 14 commits into from

Conversation

chasejohnson3
Copy link

Co-authored-by: Michal Brzus [email protected]
Co-authored-by: Benjamin Gorman [email protected]

Description

This PR is a continuation of the work that was started during ohbm brainhack 2020.
This PR allows pydra tasks to be generated from Slicer Execution Model xml files.
The generated pydra tasks are python 3.7 compatible.
The xml files included cover the BRAINSia/BRAINSTools image processing set of tools. The tasks generated by this script allowed me to create a pydra workflow with BRAINSResample and BRAINSConstellationDetector using the output volumes of BRAINSResample as the input volumes of BRAINSConstellationDetector (workflow not included in this repo).

Note: This work is a continuation from Pull Request #1 but was done in a new pull request because PR#1 came from unknown repository to which I do not have access.

Chase Johnson and others added 7 commits November 20, 2020 12:38
…, the current pydra version, and be useable by BRAINSTools (in particular, BRAINSConstellationDetector)

The following changes were made to the script:
- Removed exists as an input_spec field
- Removed hash_files as an input_spec field
- Removed variable placeholders from argstr (for example --inputVolume %s -> --inputVolume)
- Removed the spaces around the executable parameter (note, this may affect using the launcher in the future, but I have not been using a launcher."
- Removed generating __init__ files which cause relative imports to fail
…ut file name to be specified, but the filename does not already have to exist.
…ut_field outputVolume rather than inputVolume.
@chasejohnson3
Copy link
Author

@hjmjohnson

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