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

NTR: has allelopath #809

Open
mdrishti opened this issue Aug 16, 2024 · 10 comments · May be fixed by #815
Open

NTR: has allelopath #809

mdrishti opened this issue Aug 16, 2024 · 10 comments · May be fixed by #815

Comments

@mdrishti
Copy link

mdrishti commented Aug 16, 2024

The following term request is for the inverse interaction of 'allelopath of' http://purl.obolibrary.org/obo/RO_0002555. Linked to globalbioticinteractions/globalbioticinteractions#993 (comment)

Preferred term label

has allelopath

Synonyms

none

Textual definition

A relationship between organisms where one organism is influenced by the biochemical processes of another.

Additional context:
Allelopathy is a process whereby biochemicals produced by one organism positively or negatively influences growth, survival or reproduction of other organisms. See also https://www.ncbi.nlm.nih.gov/pmc/articles/PMC10172429/

Suggested parent term

(http://purl.obolibrary.org/obo/RO_0002321) -> (http://purl.obolibrary.org/obo/RO_0002437) -> (http://purl.obolibrary.org/obo/RO_0002574)

Attribution

Disha Tandon (https://orcid.org/0009-0005-5515-1230)

@nataled
Copy link
Collaborator

nataled commented Aug 19, 2024

Not an expert here, but just wondering if the second paragraph under Textual Definition is intended as part of the definition, or is just there to explain allelopathy to us non-experts? My understanding of how definitions are supposed to work is that such explanatory sentences are better as 'comment' (but I do very often see the two mixed).

@mdrishti
Copy link
Author

mdrishti commented Aug 19, 2024

@nataled , yes in this case, the 2nd paragraph of the textual definition provides additional context behind allelopathy. I plan to put this definition under 'see also' or 'comment, while describing this term.

I added it because allelopathy IMO is a broad area and in fact, in some fields like plant biology it has been thought to be a negative influence of one organism's biochemicals on another, which is not exactly right. Hence, the definition.

@nataled
Copy link
Collaborator

nataled commented Aug 19, 2024

Thanks for the clarification about the clarification ;)

I'm a bit confused by your followup statement regarding how the term has been (mis?)used in plant biology. I see a few different ways to interpret that statement:

  1. Plant biologists use 'allelopathy' to refer specifically to negative influences, but it should actually be positive influences.
  2. Plant biologists use 'allelopathy' to refer specifically to negative influences, but it should actually be positive OR negative influences.

I doubt anything that has exclusively a positive influence would be labeled with -pathy, so I'm guessing the second interpretation is correct? If so, I'd recommend that explanatory sentence be revised to say that explicitly:

"Allelopathy is a process whereby biochemicals produced by one organism positively or negatively influences growth, survival or reproduction of other organisms."

Obviously, if I'm incorrect, ignore my suggestion! But I think at least the word 'grow' should be 'growth'?

mdrishti added a commit to mdrishti/obo-relations that referenced this issue Aug 20, 2024
@nataled
Copy link
Collaborator

nataled commented Sep 9, 2024

I reviewed the changes made and see that they address my concerns. Please create the appropriate pull request and reference this issue.

@bpeters42
Copy link
Collaborator

@mdrishti - Please let us know if you have what you need to proceed.

@mdrishti
Copy link
Author

mdrishti commented Oct 7, 2024

@bpeters42, I will create a pull request soon. Just short of time these days.

@cmungall
Copy link
Contributor

cmungall commented Oct 7, 2024

Thanks everyone!

Just a general comment to help make us more responsive to community members like @mdrishti , that we in RO shouldn't require requesters to supply definitions for inverse terms, these are all generated.

(If someone could add documentation for our process that would be great! See the sparql in #51.)

Of course, we welcome any suggestions to modifications from @mdrishti to the definition for the 'canonical' direction!

It looks like the definition here for the canonical direction accidentally got copied to the seeAlso field rather than the definition field, sigh.. http://purl.obolibrary.org/obo/RO_0002555

@mdrishti
Copy link
Author

mdrishti commented Oct 8, 2024

@bpeters42, I am confused about how to assign the RO id to this term. When I first made changes to the file ro-edit.owl file (see here: mdrishti/obo-relations@3538798 ), the id got assigned as RO_0000001, which I assume is not the right way to go. I am new to ontologies and vocabularies, so I will appreciate any help in this regard.

mdrishti added a commit to mdrishti/obo-relations that referenced this issue Oct 15, 2024
@anitacaron anitacaron linked a pull request Oct 15, 2024 that will close this issue
@ddooley
Copy link
Contributor

ddooley commented Nov 4, 2024

Nov 4, 2024 call: @mdrishti people can be assigned ranges of ID, and this is held in https://github.com/oborel/obo-relations/blob/master/src/ontology/ro-idranges.owl file (or perhaps for one-offs, one can get an ID from another editor with an assigned range). This pull request already has a good ID assigned for it. But we can now simplify the definition to be in the pattern that RO has for inverse relations: "inverse of allelopath of". So any improvement of definition or comments should be placed on the main "alleopath of" relation. Can you modify the pull request accordingly?

@mdrishti
Copy link
Author

mdrishti commented Nov 6, 2024

@ddooley, yes, eventually I figured the concept of ID ranges. So, I added mine with the previous pull request

I made the changes suggested by you to the pull request now. Let me know if this looks good. Thanks.

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 a pull request may close this issue.

5 participants