GenomicsStandardsConsortium / mixs

Minimum Information about any (X) Sequence” (MIxS) specification
https://w3id.org/mixs
Creative Commons Zero v1.0 Universal
38 stars 21 forks source link

`depth` and `soil_depth` terms have same MIxS IDs #445

Open sujaypatil96 opened 2 years ago

sujaypatil96 commented 2 years ago

The depth and soil_depth terms in the terms.yaml file have the same MIxS ID i.e., MIXS:0000018.

So when rendering the new documentation pages using linkml docgen, the generator doesn't know which term to refer to to create a page for https://genomicsstandardsconsortium.github.io/mixs/0000018.

turbomam commented 2 years ago

please see https://github.com/GenomicsStandardsConsortium/mixs/issues/441 and changes made to the NMDC copy of the Google Sheets

turbomam commented 2 years ago

We should consider the MIxS IDs the primary keys for terms, not the structured comment names

lschriml commented 2 years ago

In the agriculture package, propose to update the term name from soil depth to depth. Note: Also update the definition to match grammar of other 'depth' terms.

turbomam commented 2 years ago

@lschriml , which "name" are you speaking of? Presumably Package item?

If we change the Package item or the Structured comment name, then a new ID will have to be assigned.

lschriml commented 2 years ago

In this case, the name should have been updated to just 'depth'.

When we edited for version 6.0, we made this update. the Agriculture MixS group began the creation of their package before we made this 'depth' term name update, thus they still were using the older term, soil depth. It is the same concept, the edit we will make is to the term name for soil depth to depth.

If they were defining ta new concept, then we would create a new ID. Agreed.

mslarae13 commented 1 month ago

I just checked https://genomicsstandardsconsortium.github.io/mixs/term_list/

soil_depth is not present

depth is available in Agriculture : https://genomicsstandardsconsortium.github.io/mixs/0000018/

Can we close this?