Skip to content

Update ogms.md#2912

Open
zhengj2007 wants to merge 2 commits into
masterfrom
zhengj2007-patch-1
Open

Update ogms.md#2912
zhengj2007 wants to merge 2 commits into
masterfrom
zhengj2007-patch-1

Conversation

@zhengj2007

Copy link
Copy Markdown
Contributor

Add usage

Add usage
@zhengj2007 zhengj2007 requested a review from matentzn May 23, 2026 18:55

@matentzn matentzn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is not the intention of the usage field - if you want to show usage, you should show how the ontology is used by a resource, like a database, and provide a verifiable link to that resource. A simple owl import does not in and all by itself constitute usage. There are some fringe cases where you can make an argument like that (RO, COB, OMO, etc). Is there now online database using OGMS?

@nataled

nataled commented May 26, 2026

Copy link
Copy Markdown
Contributor

I'm going to assume that the description is a roundabout way of saying that OGMS terms have been imported into the indicated ontologies, which would make the example valid. However, the key component 'url' is missing.

To fix: Replace 'type' (which doesn't appear in any example for P9) with 'url'. Then provide the IRI for an OGMS term used in one of the indicated ontologies. I'm not sure if multiple 'url' tags are allowed here but, if so, take advantage of that to provide links for each of the named ontologies. Alternatively, split the usages by ontology.

@zhengj2007

zhengj2007 commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

I intended to show that multiple ontologies import and utilize OGMS. I followed the BFO usage format as a template.

I have replaced type with url. Could you please review it again? @matentzn

set type as url
@zhengj2007 zhengj2007 requested a review from matentzn May 26, 2026 19:03
@deepakunni3 deepakunni3 self-requested a review June 9, 2026 16:27
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.

3 participants