-
Notifications
You must be signed in to change notification settings - Fork 16
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: ascatngs container #1155
fix: ascatngs container #1155
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## develop #1155 +/- ##
========================================
Coverage 99.31% 99.31%
========================================
Files 29 29
Lines 1756 1756
========================================
Hits 1744 1744
Misses 12 12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Amazing work making it work with a Dockerfile 🎖️! Well done! 💯 ⭐
I would also remove .gitmodules
and the submodules
folder here but it's up to you to create a new PR (maybe with this one as parent so we build containers only once in develop?)
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
Nice!
Great job! Thanks for the fix! : ) |
### Added: * PIP specific missing tools to config #1096 * Filtering script to remove normal variants from TIDDIT #1120 * Store TMB files in HK #1144 ### Changed: * Fixed all conda container dependencies #1096 * Changed --max_sv_size in VEP params to the size of chr1 for hg19 #1124 * Increased time-limit for sambamba_exon_depth and picard_markduplicates to 6 hours #1143 * Update cosmicdb to v97 #1147 * Updated read the docs with the changes relevant to mention #1153 ### Fixed: * Update cryptography version (39.0.1) due to security alert #1087 * Bump cryptography to v40.0.2 and gsutil to v5.23 #1154 * Pytest file saved in balsamic directory #1093 * Fix varcall_py3 container bcftools dependency error #1097 * AscatNgs container #1155
This PR:
Fixed: ascatNgs docker container replacing the container build from submodule to the Docker recipe, fixing dependencies and versions of the required tools.
Review and tests: