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

fix: remove stat files and add count files for scout upload VCFs #899

Merged
merged 12 commits into from
Apr 13, 2022

Conversation

khurrammaqbool
Copy link
Collaborator

@khurrammaqbool khurrammaqbool commented Mar 28, 2022

This PR:

Cleans up vep rules and adds stats files for all endpoint VCFs to be useful for automatic uploads of VCF files.

Added: *_counts files for each end point VCF file including *.all.filtered.pass.vcf.gz
Removed: *_stats files from vep rules

Review and tests:

  • Tests pass
  • Code review
  • New code is executed and covered by tests, and test approve

@khurrammaqbool khurrammaqbool self-assigned this Mar 28, 2022
@khurrammaqbool khurrammaqbool linked an issue Mar 28, 2022 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Mar 28, 2022

Codecov Report

Merging #899 (6bb9976) into develop (cf98015) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #899   +/-   ##
========================================
  Coverage    99.54%   99.54%           
========================================
  Files           29       29           
  Lines         1775     1775           
========================================
  Hits          1767     1767           
  Misses           8        8           
Flag Coverage Δ
unittests 99.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cf98015...6bb9976. Read the comment docs.

@khurrammaqbool
Copy link
Collaborator Author

@ivadym , I added stats files for all the filtered.passed VCF files and also added varcaller names to the output files.

@khurrammaqbool
Copy link
Collaborator Author

@ashwini06 and @ivadym, bcftools counts is added for all end point VCF files and bcftools stats is removed from vep rules.
This pull request is ready for review!

Copy link
Contributor

@ashwini06 ashwini06 left a comment

Choose a reason for hiding this comment

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

I added the comments about the count files for which we are not uploading to scout. It's up to you to either address it here or @ivadym if would like to take it in his upcoming PR. You both can decide on it :)

I wonder if this .all.filtered.pass.stats are correctly generated. Check with some dry runs. If not include the required file in

#Gather all the filtered and PASSed variants post annotation

@khurrammaqbool
Copy link
Collaborator Author

khurrammaqbool commented Apr 6, 2022

@keyvanelhami could you please share the updated list of end-point VCF files to be uploaded to scout, we can use to generate counts for?

@keyvanelhami
Copy link
Contributor

@khurrammaqbool We are using below flags and files for Scout upload. You can also look into an example of a yaml file for Scout: /home/proj/production/housekeeper-bundles/ablepoodle/2022-04-07/scout_load.yaml

vcf_cancer: /home/proj/production/housekeeper-bundles/ablepoodle/2022-04-07/SNV.somatic.ablepoodle.vardict.all.filtered.pass.vcf.gz
vcf_cancer_sv: /home/proj/production/housekeeper-bundles/ablepoodle/2022-04-07/SV.somatic.ablepoodle.manta.all.filtered.pass.vcf.gz

@khurrammaqbool
Copy link
Collaborator Author

/home/proj/production/housekeeper-bundles/ablepoodle/2022-04-07/scout_load.yaml

The list seems to include manta. We have decided to upload svdb instead. Isn't it an old list?

@keyvanelhami
Copy link
Contributor

keyvanelhami commented Apr 11, 2022

have decided to upload svdb instead. Isn't it an old list?

But is the svdb PR merged and deployed in production? If so we should open an issue so cg points to the svdb vcf instead

@khurrammaqbool
Copy link
Collaborator Author

have decided to upload svdb instead. Isn't it an old list?

But is the svdb PR merged and deployed in production? If so we should open an issue so cg points to the svdb vcf instead

It will be merged in develop and is linked to the delivery of files @ivadym is currently working with.

@khurrammaqbool khurrammaqbool changed the title fix: stat files fix: remove stat files and add count files for scout upload VCFs Apr 11, 2022
Copy link
Contributor

@ivadym ivadym left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

CHANGELOG.rst Outdated Show resolved Hide resolved
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@khurrammaqbool khurrammaqbool merged commit 096df69 into develop Apr 13, 2022
@khurrammaqbool khurrammaqbool deleted the fix/stat_files branch April 13, 2022 11:21
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.

stats files for end point VCFs
4 participants