Skip to content

Commit

Permalink
dysgu_dev <- master (#108)
Browse files Browse the repository at this point in the history
* correct usage of update_filter_value in filter_normals

It seems like at some point the usage of update_filter_value changed, because in several spots, it is missing the sample_name argument. This breaks `dysgu filter --keep-all`.

* Update main.yml

* Update main.yml

* Update requirements.txt

* updated build process to pyproject.toml (#103)

* v1.6.6 updated README

* Update main.yml

* Added pyproject.toml

* Updated pyproject.toml and setup.py

* Update pyproject.toml

---------

Co-authored-by: Dr. K. D. Murray <1560490+kdm9@users.noreply.github.com>
  • Loading branch information
kcleal and kdm9 authored Sep 12, 2024
1 parent 63b561b commit 68b8452
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 10 deletions.
4 changes: 2 additions & 2 deletions dysgu/filter_normals.py
Original file line number Diff line number Diff line change
Expand Up @@ -1025,13 +1025,13 @@ def run_filtering(args):
if mq_pri < min_mapq and mq_sup < min_mapq:
filter_results['lowMapQ'] += 1
if keep_all:
update_filter_value(r, old_filter_value, pass_prob, new_value="lowMapQ")
update_filter_value(r, sample_name, old_filter_value, pass_prob, new_value="lowMapQ")
out_vcf.write(r)
continue
if min_prob != 0 and 'PROB' in r.samples[sample_name] and r.samples[sample_name]['PROB'] < min_prob:
filter_results['lowProb'] += 1
if keep_all:
update_filter_value(r, old_filter_value, pass_prob, new_value="lowProb")
update_filter_value(r, sample_name, old_filter_value, pass_prob, new_value="lowProb")
out_vcf.write(r)
continue
if has_low_support(r, sample_name, support_fraction):
Expand Down
7 changes: 0 additions & 7 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -34,13 +34,6 @@ dependencies = [
[project.urls]
Homepage = "https://github.com/kcleal/dysgu"

[project.optional-dependencies]
test = [
"pytest",
"pytest-cov",
"pytest-mock",
]

[project.scripts]
dysgu = "dysgu.main:cli"

Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
setuptools>=63.0
cython
Cython
click>=8.0
numpy
scipy
Expand Down

0 comments on commit 68b8452

Please sign in to comment.