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

horizon AND soil_horizon ? #394

Closed turbomam closed 4 months ago

turbomam commented 2 years ago
only1chunts commented 2 years ago

to me this looks like a simple change of name accidentally let through when the soil package was created, the "soil_horizon" term should be changed to just "horizon" as it is the same MIXS ID and definition already.

Proposed action

Update soil package term name "soil_horizon" MIXSID:0001082 to "horizon" MIXSID:0001082

mslarae13 commented 6 months ago

@only1chunts @ramonawalls @turbomam can we check in about this at the call in 04/16? And add it to the CIG agenda if needed?

mslarae13 commented 6 months ago

We checked in about this on 04/16 (@only1chunts, @Woolly-at-EBI )

horizon and soil_horizon had the same ID.

To do

mslarae13 commented 4 months ago

NCBI showed 0 use of horizon 19k use of soil_horizon 8k horizon_meth/method 0 soil horizon_meth

Resolved. horizon was removed. soil_horizon is available in soil & agriculture.

turbomam commented 4 months ago

Querying NMDC's Postgres version of NCBI Biosample, generated February 26 2024

Jasper also has a JSON/MongoDB solution

select
    harmonized_name,
    count(1)
from
    ncbi_attributes_all_long naal
where
    harmonized_name = 'horizon'
    or harmonized_name = 'horizon_meth'
    or harmonized_name = 'horizon_method'
    or harmonized_name = 'soil_horizon'
    or harmonized_name = 'soil_horizon_meth'
    or harmonized_name = 'soil_horizon_method'
group by
    harmonized_name;

|harmonized_name|count |
|---------------|------|
|horizon_meth   |8,646 |
|soil_horizon   |18,799|