Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
43 of 51 tasks
Tracked by #587 ...
pvangay opened this issue Aug 4, 2022 · 15 comments
Closed
43 of 51 tasks
Tracked by #587 ...

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

pvangay opened this issue Aug 4, 2022 · 15 comments
Assignees

Comments

@pvangay
Copy link

pvangay commented Aug 4, 2022

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:

  • sample shipped amount
  • number technical replicate
  • history/extreme events
  • history/fire
  • history/flooding
  • filter method
  • experimental factor- other
  • non-microbial biomass
  • non-microbial biomass method
  • microbial biomass carbon
  • microbial biomass carbon method
  • microbial biomass nitrogen
  • microbial biomass nitrogen method
  • organic nitrogen method
  • other treatments
  • isotope exposure/addition

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
Copy link
Member

turbomam commented Aug 4, 2022

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
Copy link
Contributor

See also
#374

@ssarrafan
Copy link
Collaborator

ssarrafan commented Sep 2, 2022

Related to #373

@ssarrafan
Copy link
Collaborator

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

@ssarrafan ssarrafan removed this from the Sprint 19 milestone Oct 20, 2022
@ssarrafan
Copy link
Collaborator

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

@mslarae13
Copy link
Contributor

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

@mslarae13
Copy link
Contributor

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

@mslarae13
Copy link
Contributor

mslarae13 commented Dec 9, 2022

Progress / commit tracking

Missing guidance text and example text:
2022-12-08 PR: #571

  • sample shipped amount
  • number technical replicate
  • history/fire
  • history/flooding

2022-12-08 PR: #545

  • history/extreme events

2022-12-23 PR: #571
mixs_inspired.yaml edited directly

  • filter method
  • experimental factor- other
  • non-microbial biomass
  • non-microbial biomass method
  • microbial biomass carbon
  • microbial biomass carbon method
  • microbial biomass nitrogen
  • microbial biomass nitrogen method
  • organic nitrogen method
  • other treatments
  • isotope exposure/addition

Missing example text:
2022-12-30

  • experimental factor- other
  • replicate number
  • slope aspect (changed description to remove example)
  • extreme_unusual_properties/Al saturation
  • extreme_unusual_properties/Al saturation method
  • mean annual precipitation
  • current vegetation
  • current vegetation method
  • extreme_unusual_properties/heavy metals method
  • extreme_unusual_properties/heavy metals
  • mean seasonal precipitation
  • slope gradient
  • pH method
  • microbial biomass
  • microbial biomass method (no longer using MIxS biomass. updated MIxS inspired terms)
  • total carbon
  • total nitrogen content
  • total nitrogen content method
  • total organic carbon
  • total organic carbon method
  • salinity method
  • composite design/sieving
  • gaseous environment
  • watering regimen

Skip until discussion

  • horizon method
  • soil_taxonomic/local classification method
  • soil_taxonomic/local classification
  • soil type method
  • soil_taxonomic/FAO classification
  • link to classification information
  • link to climate information
  • water content
  • water content method
  • climate environment (I don't know how to write this according to the structure. Would like to see some completed exampels)

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

  • history/previous land use method
  • history/previous land use
  • horizon method
  • soil texture measurement
  • soil texture method

@mslarae13
Copy link
Contributor

mslarae13 commented Dec 9, 2022

Submit a GSC ticket for the following when PR approved

  • history/extreme events
  • history/fire
  • history/flooding
  • extreme_unusual_properties/Al saturation (update title and description)
  • tot_nitro vs tot_nitro_content .. content vs concentration... tot_nitro is water specific. Otherwise seems the same. Why does water need a separate slot?

@mslarae13
Copy link
Contributor

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
Copy link
Contributor

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

and horizon is commented out of mixs.yam

# horizon:

@mslarae13
Copy link
Contributor

mslarae13 commented Dec 23, 2022

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 mslarae13 moved this from In Progress to Pending Review in Squad Sprint 6: Dec 12 - Dec 22 2022 Dec 23, 2022
@mslarae13
Copy link
Contributor

@ssarrafan overdue, please add to January 2023 sprint

@mslarae13 mslarae13 moved this to 👀 In review in SubPort Squad Issues Jan 4, 2023
@ssarrafan
Copy link
Collaborator

@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
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ SubPort 1 - Done
Development

No branches or pull requests

4 participants