Skip to content

Commit

Permalink
Merge pull request #8 from Bayer-Group/fixcc
Browse files Browse the repository at this point in the history
fixed continuous coverage phenotype
  • Loading branch information
sprivite authored Nov 22, 2024
2 parents 918c9f0 + 284ffe3 commit ffb8924
Show file tree
Hide file tree
Showing 6 changed files with 56 additions and 394 deletions.
2 changes: 2 additions & 0 deletions phenex/filters/filter.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ class Filter:
but cannot trigger recursive execution. Fitlers can add columns but may not remove columns.
All classes in the filters module should subclass this class. Subclasses must implement
the _filter method.
"""

def __init__(self):
Expand All @@ -19,6 +20,7 @@ def filter(self, table: Table) -> Table:
if not set(input_columns) <= set(table.columns):
raise ValueError(f"Filter must not remove columns.")

table = table.select(input_columns)
return table

def _filter(self, table: Table) -> Table:
Expand Down
2 changes: 1 addition & 1 deletion phenex/test/phenotype_test_generator.py
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ def df_from_test_info(test_info):
if test_info.get("dates") is not None:
df[columnname_date] = test_info["dates"]
else:
test_info[columnname_date] = ibis.null(type=datetime.date)
df[columnname_date] = None

if test_info.get("values") is not None:
df[columnname_value] = test_info["values"]
Expand Down
201 changes: 0 additions & 201 deletions phenex/test/phenotypes/test_clpt_anchor_phenotype.py

This file was deleted.

Loading

0 comments on commit ffb8924

Please sign in to comment.