-
Notifications
You must be signed in to change notification settings - Fork 9
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 Fe,Al,Mn extraction procedures #166
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pvgenuchten Please submit your pull requests against develop, not master.
On the content: there are three new procedures, but no new observations, nor results (and perhaps properties). Those are necessary within the SOSA/STA framework.
3ec0ea9
to
2a66d5f
Compare
Thanks for reviewing Luis, i rebased to develop. For your second comment, is it correct to assume these aspects are managed by a script. I wanted to wait with the script until the PR had its first review. Or would you recommend to run the script now? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without observations it is not possible to correctly evaluate the introduction of these new procedures. As presented now they cannot be applied by a user of the ontology, and I cannot be sure what do they actually add anew.
Can you point me to where I can add the observations? Also I noticed the script to transform the csv runs at deploy time, so no need to run locally? glosis/.github/workflows/codelists.yml Line 39 in 7f288f1
|
@pvgenuchten @ldesousa thanks! Good to see this progress. The addition looks good although I do not fully understand @ldesousa coments. I opened this issue (#130) because typical extraction procedures for Al, Mn, Fe, Si were missing in the first version of the methods vocab we did in our vocab viewer. I ran into that when I was for looking for vocab entries for the Soils4Africa soil analyses. I had to provide URLs describing these analyses to Calisto but these were not available. The extraction methods that I referred to are commonly used methods to measure different fractions of extracted Al, Fe, Mn, Si. And are typically part of the 'standard package' of chemical soil analyses. With Ad we are currently defining observations these methods apply to. One comment from my side, the literature reference is relevant as far as I can judge, but it is not open access. We can look for another source if needed. |
Comment by Ad van Oostrum: Data using this procedures is described in https://www.isric.org/sites/default/files/ISM_SM1.pdf p10 and https://www.isric.org/sites/default/files/ISM_SM2.pdf Ad also indicates that many citations in the laboratory domain describing procedures are in closed access journals An additional challenge here is that many of these generic procedures have local differences in how they are applied, these differences are generally not described at all NRCS – Kellog lab provides a lab manual for the 3 procedures mentioned: page 620 (digital) or p. 596 (hard copy). An alternative is this isric manual chapter 12 For describing the property that is measured (‘operational definition’) only the “extraction part” is important / the most important; composition, shaking ratio (temperature), extraction time etc. Both references are then equivalent. For certain applications (criteria) it is important that you use the matching / prescribed lab method. Details in the prescription can be more important there. When describing in your database you should therefore take into account “what the user is looking for”; and that jargon is also not always clear. |
Thanks, Paul. This is helpful. I propose not to worry about 'local differences' otherwise we get nowhere with this. References to procedure manuals of well recognized laboratories (e.g. Kellog) would be fine I think. |
resolves #130