-
Notifications
You must be signed in to change notification settings - Fork 6
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
create README details tools, versions, and parameters #14
Comments
@aclum @poeli @hubin-keio is this work planned for this week or next sprint? |
@aclum @ssarrafan I am confused about "human readable description". Examples and/or scenarios will help me understand what to implement exactly. I chatted with @hubin-keio today and will have more discussions. |
example IMG annotation methods:
The make_info_file task in https://github.com/microbiomedata/mg_annotation/blob/a8c172beeb4ce93e8f8373c11e348181ade47e79/annotation_full.wdl is how I've implemented generating this file for the annotation workflow. example metatranscriptome assembly methods: The input read set was mapped to the final assembly and coverage information generated (1) MEGAHIT: An ultra-fast single-node solution for large and complex metagenomics assembly |
This request is for a readme for reads based analysis, what I saw being worked on was the reads qc process. We need both but the reads based analysis is higher priority because that is the main workflow we want to run on Bioscales and GROW for GSP. @hubin-keio |
@ssarrafan @aclum I committed the updated version
Additional output example:
|
Great thanks |
@aclum @ssarrafan I don't have permission to write centrifuge database directory. Please help move centrifuge's db_ver.info. |
@poeli is this still a problem? I see the file there. |
Moving to current sprint. Please remove from sprint if you're not actively working on this. |
Closing this per @aclum |
@poeli @hubin-keio Gottcha2 still does not list a version. Please update this. |
@aclum The issue has been resolved (output buffer didn't flush). I will proceed to rebuild the container and initiate the testing process. |
@poeli I pulled your image and pushed it back to the nmdc repo |
Updated to the master branch |
@Michal-Babins @mbthornton-lbl We'll need a new release of https://github.com/microbiomedata/ReadbasedAnalysis repo and the nmdc automation repo needs to be updated to use this new version. With this fix the expected behavior is that the |
Assets were updated to include the updated version. v1.0.5 is the current release and it now correctly reflected by the ReadbasedAnalysis.wdl and bundle.zip. Moving forward, we need to make sure any changes made to any branch, when merged to master reflect a major, minor, or patch update with the changes. |
@poeli, does this version of gottcha2 not write out ${prefix}.full.tsv or only if nothing is found? I am seeing some workflows fail because cromwell is unable to fine ${prefix}.full.tsv |
It has been determined that the sequencing workflows output a human readable description of workflow. ReadbasedAnalysis is higher priority b/c we'll need to run it on datasets for GSP. FYI @ssarrafan
The text was updated successfully, but these errors were encountered: