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

#420 added class 'soil drainage' #1274

Closed
wants to merge 3 commits into from

Conversation

KrishnaTO
Copy link
Contributor

closes #420

Copy link
Member

@cmungall cmungall left a comment

Choose a reason for hiding this comment

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

Hi @KrishnaTO

The logical definition you have chosen here is:

  • flux and characteristic-of some liquid water

  • It doesn't mention soil at all

  • is the concept of flux really the same as drainage - they seem related to me but far from equivalent

Some general guidelines that apply across all ontologies:

Additionally, some specific ENVO guidelines that may help with evaluation of your PRs in future

  • we are not using obo format as the edit format this makes PRs harder to read. Until we get a diff bot set up, it can help if you provide a description of your PR. In this case "I need soil drainage for project X. I think this should be a characteristic of soil/water, so I made a logical definition 'characteristic of some water', this will be reasoned to be under Y in the release"

@marieALaporte
Copy link
Contributor

Thanks @cmungall. We moved 'soil drainage' under 'hydrological process', and defined it as 'A hydrological process during which water moves across, through, and out of the soil as a result of the force of gravity.'
We added a subClassOf axiom 'occurs in some soil'
We need this term for the EiA initiative, where people record the soil drainage rate. We will create a PR for the rate term once this PR is accepted

@pbuttigieg
Copy link
Member

I still see this under "flux" rather than process.

@marieALaporte
Copy link
Contributor

@pbuttigieg soil drainage is under 'flux'

@marieALaporte
Copy link
Contributor

I misunderstood your comment. Will make the change

I forgot to pushed the change before....
@smrgeoinfo
Copy link

soil drainage process: a process by which water (or other liquid) is transported through a soil body. NOTE: typically gravity driven, and requires pressure gradients through connected soil porosity (e.g. permeability)
soil drainage flux: the volume of fluid transported through a soil volume per unit of time.

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 class: soil drainage, soil drainage speed
5 participants