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

Added Java Bayesian Gaussian Mixture Model backend to new annotation-based filtering tools. #8132

Draft
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

samuelklee
Copy link
Contributor

No description provided.

@samuelklee samuelklee marked this pull request as draft December 15, 2022 17:00
@gatk-bot
Copy link

Github actions tests reported job failures from actions build 3706238005
Failures in the following jobs:

Test Type JDK Job ID Logs
conda 8 3706238005.3 logs

@samuelklee
Copy link
Contributor Author

Hmm, an interesting test failure—seems like serialization of Guava ImmutableList might not be cross compatible between Java 8 and 11 (the test file attempting to be deserialized here was generated using Java 11). @cmnbroad what’s the timeline for getting rid of Java 8?

@gatk-bot
Copy link

gatk-bot commented Jan 3, 2023

Github actions tests reported job failures from actions build 3832724030
Failures in the following jobs:

Test Type JDK Job ID Logs
conda 8 3832724030.3 logs

@codecov
Copy link

codecov bot commented Jan 3, 2023

Codecov Report

Merging #8132 (28ff856) into master (757112f) will increase coverage by 0.020%.
The diff coverage is 85.791%.

Additional details and impacted files
@@               Coverage Diff               @@
##              master     #8132       +/-   ##
===============================================
+ Coverage     86.642%   86.662%   +0.020%     
- Complexity     38963     39097      +134     
===============================================
  Files           2336      2341        +5     
  Lines         182730    183522      +792     
  Branches       20066     20117       +51     
===============================================
+ Hits          158321    159043      +722     
- Misses         17366     17399       +33     
- Partials        7043      7080       +37     
Impacted Files Coverage Δ
...lkers/vqsr/scalable/ExtractVariantAnnotations.java 89.062% <ø> (-3.125%) ⬇️
.../scalable/data/LabeledVariantAnnotationsDatum.java 63.158% <ø> (-5.263%) ⬇️
...lable/modeling/VariantAnnotationsModelBackend.java 100.000% <ø> (ø)
...sr/scalable/modeling/VariantAnnotationsScorer.java 64.706% <ø> (-13.072%) ⬇️
...able/ExtractVariantAnnotationsIntegrationTest.java 98.214% <ø> (+1.548%) ⬆️
...rs/vqsr/scalable/TrainVariantAnnotationsModel.java 73.529% <45.161%> (-4.248%) ⬇️
...stering/BayesianGaussianMixtureModelPosterior.java 58.333% <58.333%> (ø)
...walkers/vqsr/scalable/ScoreVariantAnnotations.java 82.667% <66.667%> (+6.417%) ⬆️
...alable/modeling/PythonVariantAnnotationsModel.java 68.421% <66.667%> (ø)
...calable/modeling/BGMMVariantAnnotationsScorer.java 76.667% <72.000%> (+76.667%) ⬆️
... and 69 more

@gatk-bot
Copy link

gatk-bot commented Jan 3, 2023

Github actions tests reported job failures from actions build 3833183091
Failures in the following jobs:

Test Type JDK Job ID Logs
conda 8 3833183091.3 logs

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.

2 participants