Skip to content

Commit fc5cb5c

Browse files
committed
finish up separation and Corymorpha changes
1 parent 474cf24 commit fc5cb5c

5 files changed

Lines changed: 17 additions & 11 deletions

File tree

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
JAVAFLAGS=-Xmx14G
1414

1515
# Modify as appropriate
16-
WHICH=3.0draft5
16+
WHICH=3.0draft6
1717
PREV_WHICH=2.10
1818

1919
# ----- Taxonomy source locations -----
@@ -111,7 +111,7 @@ tax/ott/log.tsv: $(CLASS) make-ott.py assemble_ott.py adjustments.py amendments.
111111
bin/jython \
112112
inclusions.csv \
113113
feed/amendments/amendments-1/next_ott_id.json \
114-
tax/skel/taxonomy.tsv \
114+
tax/separation/taxonomy.tsv \
115115
feed/ott_id_list/by_qid.csv
116116
@date
117117
@rm -f *py.class

adjustments.py

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,8 @@ def deal_with_polysemies(ott):
1616
# ncbi 10197 | 6072 | Ctenophora | phylum | = comb jellies OTT 641212
1717
# ncbi 516519 | 702682 | Ctenophora | genus | = cranefly OTT 1043126
1818

19-
# The comb jellies are already in the taxonomy at this point (from skeleton).
19+
# The comb jellies are already in the taxonomy at this point (from
20+
# separation taxonomy).
2021

2122
# Add the diatom to OTT so that SILVA has something to map its diatom to
2223
# that's not the comb jellies.
@@ -28,7 +29,7 @@ def deal_with_polysemies(ott):
2829
# Diatom. Contains e.g. Ctenophora pulchella.
2930
establish('Ctenophora', ott, ancestor='Bacillariophyta', ott_id='103964')
3031

31-
# The comb jelly should already be in skeleton, but include the code for symmetry.
32+
# The comb jelly should already be in separation, but include the code for symmetry.
3233
# Contains e.g. Leucothea multicornis
3334
establish('Ctenophora', ott, parent='Metazoa', ott_id='641212')
3435

@@ -72,6 +73,10 @@ def deal_with_polysemies(ott):
7273
# not sure why this is needed.
7374
establish('Stereopsidaceae', ott, division='Fungi', source='gbif:7717211')
7475

76+
# 3.0draft5 problem
77+
establish('Corymorpha', ott, division='Cnidaria', source='ncbi:264057', ott_id='183501')
78+
establish('Corymorpha', ott, division='Nematoda', source='ncbi:364543', ott_id='860265')
79+
7580
def load_silva():
7681
silva = Taxonomy.getTaxonomy('tax/silva/', 'silva')
7782

@@ -199,7 +204,7 @@ def fix_name(bad, good):
199204
# JAR 2014-05-13 scrutinizing pin() and BarrierNodes. Wikipedia
200205
# confirms this synonymy. Dail L. prefers -phyta to -phyceae
201206
# but says -phytina would be more correct per code.
202-
# Skeleton taxonomy has -phyta (on Dail's advice).
207+
# Separation taxonomy has -phyta (on Dail's advice).
203208
silva.taxon('Rhodophyceae').synonym('Rhodophyta') # moot now?
204209

205210
silva.taxon('Florideophycidae', 'Rhodophyceae').synonym('Florideophyceae')
@@ -651,7 +656,7 @@ def patch_ncbi(ncbi):
651656
if ncbi.maybeTaxon(toplevel) != None:
652657
ncbi.taxon(toplevel).prune(this_source)
653658

654-
# - Canonicalize division names (cf. skeleton) -
659+
# - Canonicalize division names (cf. separation) -
655660
# JAR 2014-05-13 scrutinizing pin() and BarrierNodes. Wikipedia
656661
# confirms these synonymies.
657662
ncbi.taxon('Glaucocystophyceae').rename('Glaucophyta')

assemble_ott.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,7 @@ def create_ott(version):
3434
for name in names_of_interest:
3535
ott.eventLogger.namesOfInterest.add(name)
3636

37-
# idspace string 'skel' is magical, see Taxon.addSource
38-
ott.setSkeleton(Taxonomy.getTaxonomy('tax/skel/', 'skel'))
37+
ott.setSkeleton(Taxonomy.getTaxonomy('tax/separation/', 'separation'))
3938

4039
# These are particularly hard cases; create alignment targets up front
4140
adjustments.deal_with_polysemies(ott)
@@ -87,7 +86,7 @@ def merge_sources(ott):
8786
# higher priority to Worms for Malacostraca, Cnidaria so we split out
8887
# those clades from worms and absorb them before NCBI
8988
worms = adjustments.load_worms()
90-
# Malacostraca instead of Decapoda because M. is in the skeleton
89+
# Malacostraca instead of Decapoda because M. is in the separation taxonomy
9190
(malacostraca, worms_sans_malacostraca) = split_taxonomy(worms, 'Malacostraca')
9291
align_and_merge(ott.alignment(malacostraca))
9392
(cnidaria, low_priority_worms) = split_taxonomy(worms_sans_malacostraca, 'Cnidaria')

doc/method/data-package/Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ ott3.0.tgz: $(REF)/tarballs/ott3.0draft3.tgz
1414
alt-ott3.0.tgz:
1515
wget http://files.opentreeoflife.org/ott/ott3.0/ott3.0draft3.tgz
1616

17-
separation.tgz: $(REF)/tax/skel/taxonomy.tsv
18-
tar czf $@ -C $(REF) tax/skel
17+
separation.tgz: $(REF)/tax/separation/taxonomy.tsv
18+
tar czf $@ -C $(REF) tax/separation
1919

2020
silva.tgz: $(REF)/tax/silva/taxonomy.tsv
2121
tar czf $@ -C $(REF) tax/silva

inclusions.csv

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,3 +108,5 @@ Equisetidae,Equisetopsida,5662213,"issue 281"
108108
Acropora,Cnidaria,712401,"issue reftax#298"
109109
Acropora,Bryozoa,4149092,"issue reftax#298 (syn of Porina)"
110110
Dermocystidium salmonis,Ichthyosporea,5653696,"issue reftax #301"
111+
Corymorpha,Cnidaria,183501,"fallout from setting Cnidarian priority"
112+
Corymorpha,Nematoda,860265,"fallout from setting Cnidarian priority"

0 commit comments

Comments
 (0)