-
Notifications
You must be signed in to change notification settings - Fork 4
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
shortReads_assembly.wdl should output bbcms error corrected data #18
Comments
vlilanl
pushed a commit
that referenced
this issue
May 9, 2024
Shane and I tested this last week, and we had a successful run. @aclum do you want to look over it? |
@vlilanl where is the run and would you be able to do a pull request? |
This should be the run id: 8022a0d2-6443-457b-996f-de13b28cf337. I can try and do a pull request. |
vlilanl
added a commit
that referenced
this issue
Jun 4, 2024
New ticket for nmdc_automation: microbiomedata/nmdc_automation#189 |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The output of short read assembly should be included in the final output. The fastq headers do not need to be renamed but this should be included in the final workflow output and the workflow automation should make a data object for it.
This ticket is to get the WDL updated so that these bbcms task output files are get passed as outputs to the workflow, not just as outputs of the bbcms task.
The text was updated successfully, but these errors were encountered: