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

#443 added classes related to soil layer #1281

Closed
wants to merge 6 commits into from

Conversation

KrishnaTO
Copy link
Contributor

@KrishnaTO KrishnaTO commented Dec 27, 2021

Added classes:

  • soil layer
  • soil layer bottom depth
  • soil layer top depth

closes #388

Added classes:
- soil layer bottom
- soil layer bottom depth
- soil layer top
- soil layer top depth

Added synonym:
- soil layer
Added classes:
- soil profile
- soil profile characterization
AnnotationAssertion(<http://purl.org/dc/elements/1.1/creator> <http://purl.obolibrary.org/obo/ENVO_06105267> <https://orcid.org/0000-0001-6284-4821>)
AnnotationAssertion(<http://purl.org/dc/elements/1.1/date> <http://purl.obolibrary.org/obo/ENVO_06105267> "2021-12-23T04:10:36Z"^^xsd:dateTime)
AnnotationAssertion(rdfs:label <http://purl.obolibrary.org/obo/ENVO_06105267> "soil profile characterization"@en)
SubClassOf(<http://purl.obolibrary.org/obo/ENVO_06105267> <http://purl.obolibrary.org/obo/OBI_0000011>)
Copy link
Member

Choose a reason for hiding this comment

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

I am not enthused about adding more investigative processes to ENVO, there is no clear dividing line between ENVO and OBI here. But we can add contingent we may need to offload these later

Copy link
Contributor

Choose a reason for hiding this comment

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

We need that term has we need to record when the process happens. But maybe you can suggest a better way for handling time here.

Copy link
Member

Choose a reason for hiding this comment

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

I am not enthused about adding more investigative processes to ENVO, there is no clear dividing line between ENVO and OBI here.

It's not a biomedical investigation, so it's fine in ENVO. If that branch grows enough, we would spin off a new ontology for environmental assessments and investigations.

@marieALaporte
Copy link
Contributor

if having the term 'soil profile characterization process' is a blocker for this PR, we can remove it for now, as we did for 'soil profile'.
This PR is creating the following terms:

  • soil layer bottom
  • soil layer top
  • soil layer bottom depth
  • soil layer top depth

We are also proposing to add the term 'soil layer' as a synonym for 'soil horizon'

@dr-shorthair dr-shorthair self-requested a review July 29, 2022 01:53
@dr-shorthair
Copy link

dr-shorthair commented Jul 29, 2022

Hold your horses here folks.

Layers and Horizons

'soil layer' is not a synonym for 'soil horizon' (in Australia and New Zealand, at least).

  • Horizons are defined by what are broadly understood to be 'genetic' processes, or at least some compositional classification
  • Layers are more general, and are often just an arbitrary depth interval (e.g. 0.5-2 m)

profile vs sequence-of-horizons

There has been a lot of discussion around the definition of 'profile' down under as well.

There is a soil body, which is usually composed of horizons - i.e. of layers that are defined by compositional or genetic properties.

Then there is the profile, which is the set of horizons as observed at a specific site, in a core, on a section, or in the wall of a pit.

The 'profile', with its specific measurements and observations, and interpretation, is actually a sample of the soil body, or of the sequence-of-horizons. It is at least in part an artefact of the observational and sampling strategy, rather than an intrinsic representation of nature.

Copy link

@dr-shorthair dr-shorthair left a comment

Choose a reason for hiding this comment

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

@bsimons14 @abhritchie @meganrwong I suggested some alignment with ANZ here

Copy link

@dr-shorthair dr-shorthair left a comment

Choose a reason for hiding this comment

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

See comment above

@marieALaporte
Copy link
Contributor

Thanks for the feedback @dr-shorthair.
If I understand correctly the difference between a soil layer and a soil horizon, if that a soil layer might not match a soil horizon as it can be sampled at any depth.
Back to ENVO, there is currently a term 'soil horizon' which is under 'layer' and defined as 'A layer which is parallel to the soil surface and which has roughly homogenous physical, chemical and biological properties relative to the soil layers above or beneath it'. Based on the assumption that a soil horizon is a soil layer, but a soil layer is not an horizon, I propose to create a 'soil layer' class under 'layer', and to move 'soil horizon' below that class. Then 'soil layer bottom' and 'soil layer top' can be moved under this new 'soil layer' class as well. What do you think?

Regarding soil profile, we decided to not include it in this PR as we understood that it might not be in the scope of ENVO.

@marieALaporte marieALaporte self-requested a review July 29, 2022 07:26
@dr-shorthair
Copy link

Thank you @marieALaporte - yes, soil-horizon as a sub-class of soil-layer is OK.

There may (later) be an opportunity to align this with geology models, but this will be fine for now.

@marieALaporte
Copy link
Contributor

I added soil layer and moved soil horizon under it.
@dr-shorthair let me know if that looks good now. Thanks

@marieALaporte marieALaporte changed the title #443 added classes per 'soil profile' #443 added classes related to soil layer Aug 1, 2022
pbuttigieg added a commit that referenced this pull request Aug 9, 2022
@pbuttigieg pbuttigieg mentioned this pull request Aug 9, 2022
pbuttigieg added a commit that referenced this pull request Aug 9, 2022
* #388 Added classes for soil layer

Added classes:
- soil layer bottom
- soil layer bottom depth
- soil layer top
- soil layer top depth

Added synonym:
- soil layer

* #443 added classes per 'soil profile'

Added classes:
- soil profile
- soil profile characterization

* fixing equivalent axioms

* removing soil profile

* creating soil layer

* removing soil layer top and bottom as not needed

* fixing https://github.com/EnvironmentOntology/envo/runs/7749735303?check_suite_focus=true#step:4:62

Co-authored-by: Krishna Udaiwal <[email protected]>
Co-authored-by: Marie-Angélique Laporte <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New classes: soil layer, soil layer top, soil layer top depth, soil layer bottom, soil layer bottom depth
5 participants