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

Develop #12

Closed
wants to merge 62 commits into from
Closed

Develop #12

wants to merge 62 commits into from

Conversation

gconcepcion
Copy link
Contributor

Getting ready for release, merge develop into main

williamrowell and others added 30 commits May 19, 2023 17:24
- Also updated to parse_cohort:1.0.2 with an option to print version.
Avoid underestimating disk size based on first element of array.
Document submitting workflow directly to Cromwell using cURL
- updated parameter_meta
- updated inputs.json
- cleaned up some whitespace
- added comments
- using fasta filesize to estimate depth rather than a separate task; based on Greg's experiments, an uncompressed 10x FASTA is ~60GB
}
},
{
"name": "chm13v2.0",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The full name for this release is T2T-CHM13v2.0. For the abbreviated name, are we ok with something like CHM13v2, or CHM13v2p0. I'd suggest replacing the . to make pattern matching of output filenames easier.

@williamrowell
Copy link
Collaborator

de_novo_assembly.cohort.samples[].sex is unused (as far as I can tell) and can be removed from all of the inputs and documentation.

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

Successfully merging this pull request may close these issues.

3 participants