Skip to content

Changing the logic for 'input.check' files creation#124

Open
cosimolupo wants to merge 2 commits into
NeuralEnsemble:masterfrom
APE-group:PR_to_NE/enh/new_check_input_logic
Open

Changing the logic for 'input.check' files creation#124
cosimolupo wants to merge 2 commits into
NeuralEnsemble:masterfrom
APE-group:PR_to_NE/enh/new_check_input_logic

Conversation

@cosimolupo
Copy link
Copy Markdown
Contributor

As a preparatory step for CWL integration into Cobrawap as an alternative workflow manager, this PR proposes a different logic for the creation of input.check files.
The old method consisted in check_input.py blocks just reading neo files and checking for their consistency, producing no output files. Snakemake rule check_input was then explicitly asked for the creation of such files, via the shell command touch.
In the new approach, it is the check_input.py block itself that creates an empty output file named input.check, at the end of consistency checks and hence only after their successful ending. --output command-line argument is then added to such blocks. Snakemake rule check_input is changed accordingly, removing the shell command and adding the output field typically used for standard I/O blocks.
This new approach greatly helps in the I/O binding between steps in CWL workflows.

@cosimolupo cosimolupo added this to the v0.3.0 milestone Feb 9, 2026
@cosimolupo cosimolupo requested a review from mdenker February 9, 2026 16:10
@cosimolupo cosimolupo self-assigned this Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant