-
Notifications
You must be signed in to change notification settings - Fork 1
Release 0.3.0 + 0.3.1 (validated via ICA full-pipeline runs) #35
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
Merged
Merged
Changes from all commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
d7ca46b
Add option for sash sigrap individual process input to cancer report …
qclayssen f33c48c
Hypermutation Handling in PCGR (#9)
qclayssen cb5ee4d
Feature/pcgr v2.2.1 (#10)
qclayssen 9d8e4d1
fix merge duplication and imports
qclayssen 8db35b3
bump pcgr version
qclayssen c67b30a
bump gpgr version
qclayssen f9a7d3b
change print to logging
qclayssen b8ee4a0
add argument chunck size
qclayssen 8223ef9
fix version software pcgr container
qclayssen f83c23a
add logging and filter fonction to read value and replace placeholder
qclayssen 22b5c12
fix indentation
qclayssen 529309c
read gz tsv from pcgr
qclayssen d3dd182
change PCGR_CLINVAR_CLASSIFICATION
qclayssen a5fc923
Change pcgr header check
qclayssen bf930e7
tyo
qclayssen 8ea1edc
typo
qclayssen 443c5be
add missing variable pcgr_output_dir
qclayssen 966967f
Pcgr v2.2.5 (#22)
qclayssen 6a93f6b
Uptade sage vcf header check (#24)
qclayssen ed3b11b
add missing HMF_HOTSPOT constant
qclayssen a3d16f7
Fix quotation
qclayssen 9ddeac8
Fix test
qclayssen 41b83bb
remove PCGR html report from chunk (hypermutated) to reduce memory use
qclayssen c42d725
change remove pcgr parallelisation of chunks, and fix pcgr internal p…
qclayssen 03bba1d
free disk worker
qclayssen cb1c39b
remove exception to stop at fail process
qclayssen 27403cb
make thread you by pcgr constant, so in case of OOM, on the retry don…
qclayssen 2902428
Fix silent command failures by integrating error handling into execut…
qclayssen 15db538
Extract PCGR errors from log files
qclayssen 9bf22f5
Update bolt/workflows/other/cancer_report.py
qclayssen 599849d
Update bolt/common/pcgr.py
qclayssen 09a3a24
Apply suggestions from code review
qclayssen dbd500f
fix typo
qclayssen 3d2d7b1
remove duplicate fonction
qclayssen 15f0247
fix typo
qclayssen 8fa85ef
add check pcgr_hotspot_present value
qclayssen 31dabab
add is_hypermutated flag to variant counts process JSON
qclayssen f3d557c
update PCGR_TIERS_FILTERING
qclayssen 184e170
Fix PCGR hypermutated filtering, add exhaustive tests
qclayssen 5a28918
add back source comment
qclayssen 48488d6
DEV: install r-gpgr from input-sigrap-data branch for is_hypermutated…
qclayssen b97d216
Add tests for split_vcf chunking and document PCGR 500k handling
qclayssen cc39768
Fix positional arg shift in run_somatic_chunk → run_somatic call
qclayssen 7f01001
fix hypermutated tag
qclayssen 97ad9f7
Add tests for is_hypermutated dragen count fix
qclayssen b4ab71f
Separate stdout/stderr streams in execute_command
qclayssen 1b366fa
Pin r-gpgr ==2.3.0 in Dockerfile.gpgr
qclayssen f77deb4
fix doc and missing file closure
qclayssen 1b6e28a
add test
qclayssen 58987d1
change .with_suffix :
qclayssen 5949539
chore: untrack local-only files
qclayssen e0103ef
update CHANGELOG for 0.3.0 release
qclayssen 149db35
bump version to 0.3.1 and update changelog
qclayssen 358290a
ci: add smoke tests to catch image startup failures before push
qclayssen dae66e5
fix missing build and pcgr docker file
qclayssen c53557a
Fix remove unused package and log pcgr limit
qclayssen 8a35851
fix: tabix-index VCF chunks after split_vcf so PCGR can read them
qclayssen cfac848
test: assert split_vcf chunks have .tbi index (regression for PCGR v2…
qclayssen d2c390f
fix: bump r-gpgr to 2.3.1 in gpgr Docker image
qclayssen ef00deb
Merge remote-tracking branch 'origin/main' into release/0.3.1-validated
qclayssen 5baa6e1
fix: skip bcftools merge for single PCGR VCF chunk (bolt #26)
qclayssen afc46a5
style: address Stephen's review comments (bolt #35)
qclayssen 1a2a5ca
docs: document MAX_SOMATIC_VARIANTS rationale (bolt #35)
qclayssen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,3 +5,5 @@ __pycache__/ | |
| build/ | ||
| venv/ | ||
| working/ | ||
| data/ | ||
| workspace/ | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Do we have a Slack thread (or similar) to link to for this change?
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.
I believe this was an oral discussion, since PCGR's hard cap is 500k, we set 450k to avoid being right on the limit and allow margin for edge cases.
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.
I updated ADR document for sash and bolt explaining rational