microbiomedata / nmdc-schema

National Microbiome Data Collaborative (NMDC) unified data model
https://microbiomedata.github.io/nmdc-schema/
Creative Commons Zero v1.0 Universal
27 stars 8 forks source link

Add guidance and/or example text to soil template terms #382

Closed pvangay closed 1 year ago

pvangay commented 2 years ago

Some terms were reported by ESS attendees. This is an exhaustive list of terms in the Soil-EMSL template on the submission portal that either are missing guidance text and/or example text.

Missing guidance text and example text:

Missing example text:

Note that @mslarae13 will need to help with content for all of these terms, then coordinate with @turbomam to update appropriately. FYI @ssarrafan

turbomam commented 2 years ago

Wow! That's a lot! Adding annotations to those slots will make the schema much better!

Many of those slots come from MIxS. Ideally any annotations that are prepared for them would go straight to the GSC. If there's a concern that the turn around time would be too slow, I recommend that @mslarae13 or any other contributors make a copy of the MIxS 6 term updates Google Sheets and enter their annotations there. It will be easy to do a diff between our version ans their version. If they decide to accept them, then it will already be in a suitable format. If they reject them or if the process starts dragging on too long, it will be easy to feed the annotations into sheets-for-nmdc-submission-schema, the input for our submission portal schema.

Here's some notes about the LinkML slots that currently appear in the DH help dialogs, which include the catch-all "Guidance" section.

mslarae13 commented 2 years ago

See also https://github.com/microbiomedata/nmdc-schema/issues/374

ssarrafan commented 2 years ago

Related to https://github.com/microbiomedata/nmdc-schema/issues/373

ssarrafan commented 1 year ago

@mslarae13 do you want this issue in the next sprint? Next sprint is very short due to the retreat.

ssarrafan commented 1 year ago

@mslarae13 I'm removing this one from the sprint until there is a plan to work on it.

mslarae13 commented 1 year ago

@ssarrafan can you add this to the sprint starting Monday?

mslarae13 commented 1 year ago

I'm using "comments" in LinkML to capture what should be in "guidance" on the submission portal

mslarae13 commented 1 year ago

Progress / commit tracking

Missing guidance text and example text: 2022-12-08 PR: https://github.com/microbiomedata/nmdc-schema/pull/571

2022-12-08 PR: https://github.com/microbiomedata/nmdc-schema/pull/545

2022-12-23 PR: https://github.com/microbiomedata/nmdc-schema/pull/571 mixs_inspired.yaml edited directly

Missing example text: 2022-12-30

Skip until discussion

Commented out? (How are these commented out but still in the submission portal if the submission portal schema is made from these files?

mslarae13 commented 1 year ago

Submit a GSC ticket for the following when PR approved

mslarae13 commented 1 year ago

For experimental_factor_other , from Mark Instead of "this is open text" I would recommend "This column (or field) accepts open-ended text about your sample. We recommend using key:value pairs. Then your keys will be considered for inclusion as future slots/terms in this data collection template."

then some examples could be

value: "sound: oink" value: "flavor: orange; calories: 100"

mslarae13 commented 1 year ago

@turbomam Do you know why horizon and horizon method are commented out in nmdc.yaml

and horizon is commented out of mixs.yam https://github.com/microbiomedata/nmdc-schema/blob/c097bfdbf5feae5b1895deb3079f4d2b27e02edf/src/schema/mixs.yaml#L6437

mslarae13 commented 1 year ago

Is adding an example where one does not exists count as changing the slot & it "no longer being the same IRI"? Per the conversation at the metadata sync : https://docs.google.com/document/d/1CDLw4zwD4Z_0dNHyT8nQGey6fDvVd0zQd4cDqG_ulqg/edit?usp=sharing

@turbomam , @cmungall , @sujaypatil96 thoughts?

mslarae13 commented 1 year ago

@ssarrafan overdue, please add to January 2023 sprint

ssarrafan commented 1 year ago

@ssarrafan overdue, please add to January 2023 sprint

@mslarae13 Are these still in review status? I will move to the next sprint but let me know if it should go to the backlog instead.

mslarae13 commented 1 year ago

PR has been merged. Any outstanding slots to update have individual issues assigned. Closing this one.