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

Add intentional pre and post checks for file emptiness #177

Open
a-frantz opened this issue Aug 7, 2024 · 0 comments
Open

Add intentional pre and post checks for file emptiness #177

a-frantz opened this issue Aug 7, 2024 · 0 comments

Comments

@a-frantz
Copy link
Member

a-frantz commented Aug 7, 2024

It's a relatively common standard in the bioinformatics tools we wrap to not have any special handling for empty inputs or outputs (including headered files without content/alignments). These "empty" files can currently be passed around between many of our tools resulting in a bunch of no-ops and a tough-to-track class of errors.

We should have lightweight checks before and after the "main computation" of our tasks to ensure that only "populated" files are being processed. We'll need a short investigation to find out which tools this wrapping will be needed for, as it would likely be redundant with some tools that error as one might expect on empty inputs and outputs.

This is related to #172 , and closing this issue should probably close #172 as well.

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

No branches or pull requests

1 participant