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 soil_conductivity slot descriptions #826

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mslarae13
Copy link
Contributor

@mslarae13 mslarae13 commented Jul 17, 2024

#813

Add descriptions for these slots:

  • warning Slot 'soil_conductivity' does not have recommended slot 'description' (recommended)

@mslarae13 mslarae13 linked an issue Jul 17, 2024 that may be closed by this pull request
@@ -12287,6 +12287,7 @@ slots:
interpolated: true
partial_match: true
soil_conductivity:
description: Soil electrical conductivity (EC) is a measure of how well soil water can carry an electrical current. Electrical conductivity of soil solution is a reliable indicator of its solute (cation or anion) concentration with 1 dS/m approximately equivalent to 10 meq/L (U.S. Salinity Laboratory Staff, 1954).
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@only1chunts @lschriml
This slot was added by "The AgMicrobiome project, JP Dundore-Arias, Linda Kinkel"
Do you have a way to contact someone to check in that this description fits their goal for this slot?

@mslarae13 mslarae13 changed the title add missing slot descriptions add soiil_conductivity slot descriptions Jul 17, 2024
@mslarae13 mslarae13 changed the title add soiil_conductivity slot descriptions add soil_conductivity slot descriptions Jul 17, 2024
@lschriml
Copy link
Member

lschriml commented Jul 18, 2024 via email

@mslarae13
Copy link
Contributor Author

@lschriml

If the previous definition was "conductivity of some soil" that somehow got lost as there's no description now!
We can make the update to just include the original definition, however, I would argue and have a strong opinion against definitions that re-use the word. That's not a definition/description and isn't helpful.

conductivity is the conductivity means nothing to someone who is looking for a description.

I found this description by looking into what is soil conductivity. It might not be right! And that's why I want to confirm with those that created the extension. But I do think we should provide a better and more informative description.

@lschriml
Copy link
Member

lschriml commented Jul 19, 2024 via email

@lschriml
Copy link
Member

lschriml commented Jul 20, 2024 via email

@mslarae13 mslarae13 self-assigned this Sep 3, 2024
@mslarae13
Copy link
Contributor Author

@lschriml

I changed the description back to the original description.
I moved my clarification about what soil conductivity really is to a comment. I have a strong opinion about definitions that just re-use the word they're defining! It's bothers me 😅

Is this sufficient? So I brought back the original description that got lost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Schema elements should have descriptions
2 participants