aboutsummaryrefslogtreecommitdiff
path: root/scripts/rqtl2
diff options
context:
space:
mode:
Diffstat (limited to 'scripts/rqtl2')
-rw-r--r--scripts/rqtl2/phenotypes_qc.py11
1 files changed, 5 insertions, 6 deletions
diff --git a/scripts/rqtl2/phenotypes_qc.py b/scripts/rqtl2/phenotypes_qc.py
index 4191df9..3448790 100644
--- a/scripts/rqtl2/phenotypes_qc.py
+++ b/scripts/rqtl2/phenotypes_qc.py
@@ -169,7 +169,7 @@ def merge_dicts(*dicts):
return reduce(lambda merged, dct: {**merged, **dct}, dicts, {})
-def decimal_points_error(
+def decimal_points_error(# pylint: disable=[too-many-arguments]
filename: str,
rowtitle: str,
coltitle: str,
@@ -178,7 +178,7 @@ def decimal_points_error(
decimal_places: int = 1
) -> Optional[InvalidValue]:
"""Returns an error if the value does not meet the checks."""
- if not bool(decimal_places_pattern(1).match(cellvalue)):
+ if not bool(decimal_places_pattern(decimal_places).match(cellvalue)):
return InvalidValue(filename, rowtitle, coltitle, cellvalue, message)
return None
@@ -188,8 +188,7 @@ def integer_error(
rowtitle: str,
coltitle: str,
cellvalue: str,
- message: str,
- decimal_places: int = 1
+ message: str
) -> Optional[InvalidValue]:
"""Returns an error if the value does not meet the checks."""
try:
@@ -201,7 +200,7 @@ def integer_error(
return InvalidValue(filename, rowtitle, coltitle, cellvalue, message)
-def qc_pheno_file(
+def qc_pheno_file(# pylint: disable=[too-many-arguments]
filepath: Path,
samples: tuple[str, ...],
phenonames: tuple[str, ...],
@@ -281,7 +280,7 @@ def run_qc(# pylint: disable=[too-many-arguments]
logger.error("We found the following errors:\n%s",
"\n".join(f" - {error}" for error in errors))
return 1
- # TODO: Run QC on actual values
+ # Run QC on actual values
# Steps:
# - Extract file to specific directory
extractiondir, *_bundlefiles = extract_bundle(phenobundle, workingdir)