Skip to content

Commit f20da7e

Browse files
authored
Merge pull request #37 from harmsm/main
bugfix: right response to seed species not found on ott
2 parents aa0f673 + defcec8 commit f20da7e

File tree

5 files changed

+14
-10
lines changed

5 files changed

+14
-10
lines changed

docs/badges/coverage-badge.svg

Lines changed: 1 addition & 1 deletion
Loading

docs/badges/ghwf.svg

Lines changed: 7 additions & 7 deletions
Loading

docs/badges/tests-badge.svg

Lines changed: 1 addition & 1 deletion
Loading

topiary/io/seed.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,9 +121,12 @@ def read_seed(df,
121121
w += f" '{b}'\n"
122122
warnings.warn(w)
123123

124+
125+
tmp_ott_list = [ott for ott in ott_list if ott is not None]
126+
124127
# Figure out if the default is to reconcile or not based on taxonomic
125128
# distribution of alignment.
126-
mrca = topiary.opentree.ott_to_mrca(ott_list=ott_list,
129+
mrca = topiary.opentree.ott_to_mrca(ott_list=tmp_ott_list,
127130
avoid_all_life=True)
128131

129132
if mrca["is_microbial"]:

topiary/opentree/util.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -438,6 +438,7 @@ def ott_to_mrca(ott_list=None,
438438
out["ott_rank"] = "no rank"
439439
out["lineage"] = None
440440
out["taxid"] = 1
441+
out["is_microbial"] = False
441442

442443
return out
443444

0 commit comments

Comments
 (0)