-
Notifications
You must be signed in to change notification settings - Fork 23
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
Fixing critical typo in HaplotypeCaller disk spec. #450
Conversation
Thank you for the great work on these updates. I tested this branch on 3,000 malaria samples with the SRFlowcell workflow on Terra and it worked perfectly. Review Summary of Key Improvements
|
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.
Please check the conversation for the details.
- added option to use gnarly genotyper - added het inputs to joint genotyping - fixed java memory allocation in joint genotyping to be based on memory of the VM, not hard-coded
- Added stack trace logging for errors in `ExtractVariantAnnotations`, `TrainVariantAnnotationsModel`, and `ScoreVariantAnnotations`.
…tion to reflect the actual name.
- Removed `HAPCOMP`, `HAPDOM`, and `HEC` from default annotations for SNP and INDEL VETS filtration. Need to do more testing / debugging to include these in joint calling. - Fixed name of outdir in `ConvertToZarrStore` to be correct for this workflow. - Updated the zarr conversion to use parallel Dask processes and to log to stdout.
No description provided.