microbiomedata / submission-schema

https://microbiomedata.github.io/submission-schema/
MIT License
1 stars 1 forks source link

Consolidate storage condition terms #212

Open pvangay opened 2 years ago

pvangay commented 2 years ago

From MIxS

I thought that there was a decision to break up this term into two terms? I presume, storage temperature and storage duration?

Please review @mslarae13 and reassign to @turbomam once next steps are determined.

turbomam commented 2 years ago

This makes sense and I will wait for @mslarae13's feedback.

I check several different resources when researching a DH issue like this. Apologies if you already know all of this!

In this case, I have prepared a summary of all what I believe to be all data.tsv rows relating to sample storage. It also confirms the provenance of each term. The necessary and sufficient step was filtering the label column on 'stora'. I also looked for 'store' but didn't get any hits. Searching for 'stor' also brings up all of the 'history' columns.

Ontology ID label
emsl:EMSL_store_temp EMSL Sample Storage Temperature, deg. C
mixs_modified:storage_condt storage conditions
MIXS:0000110 sample storage temperature
mslarae13 commented 2 years ago

I see "sample storage temperature" and "EMSL Sample Storage Temperature, deg. C" in the DH interface -- these should be consolidated into one.

I see that "storage conditions" is still in the DH interface and appears to be an enumeration. I don't think this is correct usage of this term and likely resulted from the attribute name (see above). My suggestion is to remove this term. Do we need a new term to represent the sample storage duration? Did I miss this in the DH interface?

pvangay commented 2 years ago

Ok what I'm reading here is that one action is to: remove "EMSL Sample Storage Temperature, deg. C" and keep "sample storage temperature".

That leaves "storage conditions". Right now, "storage conditions" and "sample storage temperature" are both in MixS modified. Do we need to remove "storage conditions" and replace it with a "sample storage duration"? @mslarae13