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

Obsolete behavioral control of excretion #158

Merged
merged 7 commits into from
Mar 19, 2023
Merged

Obsolete behavioral control of excretion #158

merged 7 commits into from
Mar 19, 2023

Conversation

DitchingIt
Copy link
Collaborator

Fixes #115

Removed equivalence axiom from phenotype branch.
Move defecation and urination to subclasses of emission.
Deprecated Excretion.
@matentzn
Copy link
Collaborator

matentzn commented Feb 28, 2023

Sorry @DitchingIt - I am not the right person for reviewing content changes to any ontology (I have a rule: don't look at content - else I will never be able to do any work); I would recommend @sbello or @rays22 as reviewers

@DitchingIt DitchingIt requested review from rays22 and removed request for matentzn February 28, 2023 12:52
@DitchingIt DitchingIt self-assigned this Feb 28, 2023
Copy link
Collaborator

@rays22 rays22 left a comment

Choose a reason for hiding this comment

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

Please, do not obsolete NBO:0000038 behavioral control of excretion.
Instead, remove the axiom

Fo further explanation, please find my comments here.

@sbello
Copy link

sbello commented Mar 1, 2023

Agree with @rays22 just remove the axiom. Behavioral control of excretion is still a valid term even if GO is eliminating the process excretion.

@DitchingIt DitchingIt requested a review from rays22 March 2, 2023 06:36
@DitchingIt
Copy link
Collaborator Author

Hi @rays22
Not sure if you've seen my resubmitted PR or have I made a mistake in the process?
Thanks

Copy link
Collaborator

@matentzn matentzn left a comment

Choose a reason for hiding this comment

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

This is ready to merge!

@DitchingIt DitchingIt removed the request for review from rays22 March 19, 2023 17:50
@matentzn matentzn merged commit 7659a4d into master Mar 19, 2023
@matentzn matentzn deleted the Excretion branch March 19, 2023 18:03
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.

obsolete: behavioral control of excretion
4 participants