Make SASID an optional parameter (!11) · Merge requests · ResearchAndDevelopment / preprocessing-cwl · GitLab

Snippets Groups Projects

Merged Mick Veldhuis requested to merge make-sasid-optional into main 7 months ago

All threads resolved!

In the current implementation the SASID is a required workflow input. This makes no sense when the pipeline hasn't been called by a SAS process. Hence, this MR makes the SASID parameter optional; instead of using the SASID in name of the output MSs, the Observation ID is reused.

Activity

Mick Veldhuis assigned to @veldhuis 7 months ago

assigned to @veldhuis
Mick Veldhuis added 3 commits 7 months ago
added 3 commits

85a146f8 - Remove redudant empty line

35d3fd63 - Remove redundant type check

40b15b61 - Add JSON with bare minimum paramameters

Compare with previous version
Mick Veldhuis marked this merge request as ready 7 months ago

marked this merge request as ready
Matthijs van der Wild @wild started a thread on the diff 7 months ago

Resolved 7 months ago by Mick Veldhuis
Last reply by Mick Veldhuis 7 months ago

Matthijs van der Wild @wild started a thread on the diff 7 months ago

Resolved 7 months ago by Mick Veldhuis

Matthijs van der Wild requested review from @wild 7 months ago

requested review from @wild

Matthijs van der Wild approved this merge request 7 months ago

approved this merge request

Mick Veldhuis resolved all threads 7 months ago

resolved all threads

Mick Veldhuis merged 7 months ago

merged

Mick Veldhuis mentioned in commit 7 months ago

mentioned in commit 0964b072

Please register or sign in to reply