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

feat: release 9.0.0 #902

Merged
merged 71 commits into from
May 3, 2022
Merged

feat: release 9.0.0 #902

merged 71 commits into from
May 3, 2022

Conversation

ashwini06
Copy link
Contributor

@ashwini06 ashwini06 commented Apr 5, 2022

This PR:

Creates a new release of v9.0.0

Added:
^^^^^^

Changed:
^^^^^^^^

Fixed:
^^^^^^

Removed
^^^^^^^

To Do on CG
^^^^^^^^^^

Review and tests:

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

ashwini06 and others added 30 commits December 13, 2021 13:16
Conflicts:
	CHANGELOG.rst
* feat: add canfam3 reference files

* feat: add canfam3 reference files, fix test

* feat: add canfam3 reference files, fix test

* modify CHANGELOG.rst with updates

* snakefile is assigned in initialize

* Snakefile is not defined in as a -s cli option but as following the call of init and --genome-version

* black linting

* Update BALSAMIC/constants/reference.py

Co-authored-by: ashwini06 <ashwini06@users.noreply.github.com>

* Update BALSAMIC/constants/reference.py

Co-authored-by: ashwini06 <ashwini06@users.noreply.github.com>

* Update BALSAMIC/utils/cli.py

Co-authored-by: ashwini06 <ashwini06@users.noreply.github.com>

* Update BALSAMIC/workflows/reference-canfam3.smk

Co-authored-by: Hassan Foroughi <hassanfa@users.noreply.github.com>

* implemented comments

* implemented comments, linted

* functions get_md5 and create_md5 in utils/cli.py and tests for cosmic key provided for non canfam3 genome-version

* linting

* add tests for get_md5 and create_md5

* change md5 to sha512 and cosmic key log error

* modify md5 return to sha512

* linting

* slight change in formatting

* sha512 to crc32/md5

* Update BALSAMIC/workflows/reference-canfam3.smk

Co-authored-by: ashwini06 <ashwini06@users.noreply.github.com>

* Update BALSAMIC/workflows/reference-canfam3.smk

Co-authored-by: ashwini06 <ashwini06@users.noreply.github.com>

* hash->hashed and remove sequencing_type as input for get_snakefile

* remove unused sequencing_type

Co-authored-by: ashwini06 <ashwini06@users.noreply.github.com>
Co-authored-by: Hassan Foroughi <hassanfa@users.noreply.github.com>
feat: report umi duplication rate
* version independent panel bed names
* default conditions for panel qc validation
@codecov
Copy link

codecov bot commented Apr 5, 2022

Codecov Report

Merging #902 (a376a1f) into master (9578e97) will decrease coverage by 0.35%.
The diff coverage is 99.28%.

@@            Coverage Diff             @@
##           master     #902      +/-   ##
==========================================
- Coverage   99.61%   99.25%   -0.36%     
==========================================
  Files          29       29              
  Lines        1816     1749      -67     
==========================================
- Hits         1809     1736      -73     
- Misses          7       13       +6     
Flag Coverage Δ
unittests 99.25% <99.28%> (-0.36%) ⬇️

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

Impacted Files Coverage Δ
BALSAMIC/constants/variant_filters.py 100.00% <ø> (ø)
BALSAMIC/constants/workflow_params.py 100.00% <ø> (ø)
BALSAMIC/constants/workflow_rules.py 100.00% <ø> (ø)
BALSAMIC/commands/run/analysis.py 99.05% <66.66%> (-0.95%) ⬇️
BALSAMIC/__init__.py 100.00% <100.00%> (ø)
BALSAMIC/commands/config/base.py 100.00% <100.00%> (ø)
BALSAMIC/commands/config/case.py 96.42% <100.00%> (+0.06%) ⬆️
BALSAMIC/commands/config/qc.py 100.00% <100.00%> (ø)
BALSAMIC/commands/init/base.py 100.00% <100.00%> (ø)
BALSAMIC/commands/report/deliver.py 100.00% <100.00%> (+0.85%) ⬆️
... and 11 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 9578e97...a376a1f. Read the comment docs.

ashwini06 and others added 9 commits April 6, 2022 13:40
* remove MSK_impact files from config

* remove strelka and pindel from config analysis

* remove strelka and pindel from balsamic env variables

* remove strelka,mutect and baserecalibator from cluster json

* remove strelka and pindel from constants

* update changelog
* update changelog

* update changelog

* remove bcftools_stats

* add bcftools_stats

* update changelog

* add bcftools +counts to end point VCFs

* update black linter

* remove bcftools counts for VCFs not to be uploaded to scout

* remove bcftools counts for VCFs not to be uploaded to scout

* update changelog
* add balsamic method description

* add svdb to softwares

* update changelog

* adding more text

* add wgs sentieon descr

* add sentieon filter text

* add spell and grammer checks

* update changelog

* spell and grammer checks for balsamic filters

* small text replacemnets

* add svdb citation

* fix review comments-1

* fix review comments-2

* fix review comments-3

* edit methods text

* few more text edits
* update changelog

* add ascatngs copynumber file

* add ascat to copynumber output

* fix fpdf2 to version 2.4.6
@ashwini06 ashwini06 changed the base branch from master to develop April 20, 2022 07:00
@ashwini06 ashwini06 changed the base branch from develop to master April 20, 2022 07:01
rannick and others added 11 commits April 20, 2022 10:36
* feat: add qc only workflow

* conftest qc config

* formatting

* fixing code smells and attemps to reduce duplication

* formatting

* remove qc_config container version

* formatting

* fix qc test

* fix qc test

* test fixing

* test fixing

* formatting

* draft pytest qc

* feat: add qc only workflow

* conftest qc config

* formatting

* fixing code smells and attemps to reduce duplication

* formatting

* remove qc_config container version

* formatting

* fix qc test

* fix qc test

* test fixing

* test fixing

* formatting

* draft pytest qc

* black linting + removing test_config_qc_graph_value_error

* add container version in qc

* add tests for QC graph generation and ValueError

* black

* upgrade black because of click update and remove unused config_dick in conftest

* update black in github action

* remove benchmark plot for qc

* address duplication

* changelog and conftest update

* Apply suggestions from code review

Co-authored-by: ashwini06 <ashwini06@users.noreply.github.com>

* balck upgrade in changelog

* remove variable germline_call_samples as only qc

* Update BALSAMIC/workflows/QC.smk

Remove chromlist

Co-authored-by: ashwini06 <ashwini06@users.noreply.github.com>

* Revert "changelog and conftest update"

This reverts commit fabddaf.

* remove umiworkflow and vcf from qc

* remove bedchrom from qc workflow

* update to CHANGELOG from black version

* add canfam3 to workflow qc test

* remove unused import

* remove wgs from qc workflow and modify command to qc_panel

* add qc_panel to click

* adapt conftest to qc_panel

* adapt  to qc_panel

* use qc_panel also in get_snakemake instead of only qc and models.py comments

* modidy test_utils to qc_panel

* qc metrics

* keep chr in refGene as ref fasta uses chr

* add more memory to picard for canfam3 because dogs have 38 chromosomes

* no analysis_specific_results in QC.sml

* stop removing chr with canfam3 references

Co-authored-by: ashwini06 <ashwini06@users.noreply.github.com>
* add ascat to copynumber file

* update changelog

* add balsamic annotation resources to docs

* update changelog

* fix review suggestion

* modify description for CLNACC

* modify description for COSMIC_CNT

* modify description for COSMIC_CNT
* update delly to 0.9.1

* update changelog

* update changelog

* fix Dockerfile

* fix Dockerfile

* remove OMP

* add OMP

* remove OMP

* add OMP
* add new gnomad pop freq for umi workflow

* edit balsamic filters docs

* update changelog

* fix ident

* fix typo

* add pop_freq_umi to model attributes

* fix indentation

* add review suggestions

* add PR number to changelog
* update changelog

* add tumor only cnv analysis and fix messeges in rules

* fix messege text

* update BALSAMIC documentation
* update changelog

* update changelog

* add copy-number profile to delly tumor only

* add copy-number profile to delly tumor only
* remove gatk haplotypecallers

* remove haplotypecaller and tnsnv from cluster json

* remove haplotypecaller from models

* remove haplotypecaller from workflow params

* remove haplotypcaller and tnsnv from analysis json

* remove unused callers from tests

* update changelog
* update align_qc base image

* update align_qc tool versions

* add tabix version

* remove csvkit from align_qc

* remove csvkit frm bioinfo_tool env

* update align _qc container tool versions in readthedocs

* add samtools versions to tests

* update changelog

* update base image in coverage_qc container

* update tool versions in cover_qc container

* update tool versions in bioinfo softwares docs

* update changelog

* update base image in container varcall_cnvkit

* update cnvkit version

* update purecn version and lock bcftools and tabix versions

* update docs and changelog

* update base image in varcall_py36 container

* update tools in varcall_py36

* update samtools version in docs

* update changelog

* update base image of annotate container

* update ensembl vep in annotate container

* update readthedocs for vep version

* update changelog

* fix typo in varcall_py27
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 2, 2022

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
0.0% 0.0% Duplication

@ashwini06 ashwini06 marked this pull request as ready for review May 2, 2022 14:49
@ashwini06 ashwini06 merged commit abf01aa into master May 3, 2022
@ashwini06 ashwini06 deleted the release/9_0_0 branch May 3, 2022 11:31
@ivadym ivadym added the Release Release related changes label Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Release related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants