-
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
Poc/methylation #143
base: main
Are you sure you want to change the base?
Poc/methylation #143
Conversation
Quality Gate failedFailed conditions 67.5% Coverage on New Code (required ≥ 80%) |
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.
The build failed with a Critical, Major and Minor code issue: https://sonarcloud.io/project/issues?id=molgenis_vip-report&pullRequest=143&resolved=false&inNewCodePeriod=true as well as test coverage being too low.
Failed conditions
[67.5% Coverage on New Code](https://sonarcloud.io/component_measures?id=molgenis_vip-report&pullRequest=143&metric=new_coverage&view=list) (required ≥ 80%)
@@ -33,10 +33,14 @@ usage: java -jar vcf-report.jar -i <arg> [-o <arg>] [-f] [-t <arg>] [-pb | |||
-g,--genes <arg> Genes file to be used as reference track in the | |||
genome browser, UCSC NCBI RefSeq GFF file | |||
(gff.gz or gff3.gz). | |||
-c,--cram <arg> Comma-separated list of sample-bam files | |||
-c,--cram <arg> Comma-separated list of sample-cram files |
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.
sharp
} | ||
|
||
private static boolean isFeature(List<ContigInterval> contigIntervals, BedmethylLine bedmethylLine) { | ||
for (ContigInterval contigInterval : contigIntervals) { |
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.
a more efficient solution would be to create a map from Contig to ContigInterval before looping through .bed lines. this would speed up the slicing of the bedmethyl
return false; | ||
} | ||
|
||
private static boolean isOverlappingFeature(BedmethylLine bedmethylLine, ContigInterval contigInterval) { |
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.
would have been nice to refactor the code copied from GenesFilter.java
in order to reuse this method.
No description provided.