Skip to content

Commit 33fd1d9

Browse files
committed
Merge branch 'dev' of github.com:broadinstitute/seqr-loading-pipelines into benb/i_guess_this_is_sufficient
2 parents d23f441 + c30d90f commit 33fd1d9

File tree

2 files changed

+1
-7
lines changed

2 files changed

+1
-7
lines changed

v03_pipeline/lib/misc/family_entries.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,7 @@ def remove_family_guids(
115115
if len(family_indexes_to_keep) > 0
116116
else hl.empty_array(ht.family_entries.dtype.element_type),
117117
)
118+
ht = ht.filter(hl.any(ht.family_entries.map(hl.is_defined)))
118119
return ht.annotate_globals(
119120
family_guids=ht.family_guids.filter(
120121
lambda f: ~family_guids.contains(f),

v03_pipeline/lib/misc/family_entries_test.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -273,13 +273,6 @@ def test_remove_family_guids(self) -> None:
273273
],
274274
],
275275
),
276-
hl.Struct(
277-
id=1,
278-
filters={'HIGH_SR_BACKGROUND'},
279-
family_entries=[
280-
None,
281-
],
282-
),
283276
],
284277
)
285278

0 commit comments

Comments
 (0)