-
Notifications
You must be signed in to change notification settings - Fork 0
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
🚧 update tumor only workflow #16
Conversation
bcba36e
to
f7773fb
Compare
f7773fb
to
0637980
Compare
🔧 update readme and pubapp format update docker table 🔧 update readme and pubapp format update docker table 📚 update pubapp styling
📚 update filetypes 🔧 add suggested blacklists/PON
17bada5
to
ebdb1bd
Compare
Co-authored-by: Miguel Brown <miguel.a.brown@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Could look into replacing verbatim tool copies with raw git links, but at this point, probably not worth fawning over. Do we want to throw in the release version now, or later?
Yeah probably worth looking into during the strategic week next month. Perhaps we should make a ticket @migbro
Later. I think there's a couple more things that we'll need to go over like workflow/output naming. |
Description
Part of https://d3b.atlassian.net/browse/BIXU-2125
Part of https://d3b.atlassian.net/browse/BIXU-3672
Filtration Updates
gatk_filter_name
andgatk_filter_expression
valuesWorkflow Optimization
GNU parallel
samtools
implementation; no longer performing pileup on ALT contigscalling_regions
,blacklist_regions
, andcnv_blacklist_regions
which are then processed byprepare_regions.cwl
awk_chrlen_builder.cwl
,gatk_bedtointervallist.cwl
,gatk_intervallisttobed.cwl
, andgatk_intervallisttools.cwl
samtools calmd
now only runs when the input is CRAM or if user specifiesrun_calmd_bam
Quality of Life
[2,3,4]
tocfree_ploidy
FASTA
,FAI
, andDICT
indexed_reference_fasta
,gem_mappability_file
,panel_of_normals
,blacklist_regions
, andcnv_blacklist_regions
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist: