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

Add information about the identifiers used by Mondo to the README #7478

Merged
merged 7 commits into from
May 19, 2024

Conversation

gaurav
Copy link
Contributor

@gaurav gaurav commented Mar 25, 2024

I noticed that the information on constructing a Mondo identifier is currently located in two places:

  1. http://obofoundry.org/id-policy says that the URI "http://purl.obolibrary.org/obo/" IDSPACE "_" LOCALID should be used.
  2. https://mondo.readthedocs.io/en/latest/editors-guide/f-entities/ says that "All classes are in MONDO ID space"

I really like how the LOINC User Guide (in section 12.7.2) clearly describes how to construct an RDF URI for a LOINC concept, and I think it would be great to include this information in Mondo's documentation somewhere.

In this PR I propose a short paragraph on the main README page in a new section entitled "Identifiers" that would contain this information. If there is a better place to place this document (e.g. the editor's guide linked to above), let me know and I'll make a PR to target that file itself. Feel free to change my proposed wording or move it another file if that would be better.

Copy link
Member

@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 content is correct and very much appreciated, but it needs to go into the "Mondo user guide". I can only find an editors guide and a developer guide.

@nicolevasilevsky What would be the best place to put such content? Should we start building a "user guide"?

@nicolevasilevsky
Copy link
Member

nicolevasilevsky commented Mar 29, 2024

on our readthedocs documentation, we actually have a "user's corner" on the left hand nav, but I have my doubts if many users know that is there. Screenshot pasted below

image

I've always thought the website was meant to be more of the user documentation. But then there was talk at some point about merging the website and the readthedocs site, which I am in favor of.

I think a more prominent users guide is a great idea.

At C-Path, we are going to have a week-long hackathon in a few weeks where we are going to focus on documentation. Maybe Mondo should consider a 1 or 2 day hackathon to focus on revising + updating documentation and aggregating and reorganizing all of the docs?

Copy link
Collaborator

@twhetzel twhetzel left a comment

Choose a reason for hiding this comment

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

small change requested

README.md Outdated Show resolved Hide resolved
@gaurav
Copy link
Contributor Author

gaurav commented Apr 4, 2024

My FHIR RDF colleagues pointed out one other thing I missed in addition to Trish's feedback:

@gaurav
Copy link
Contributor Author

gaurav commented May 2, 2024

@nicolevasilevsky:

on our readthedocs documentation, we actually have a "user's corner" on the left hand nav, but I have my doubts if many users know that is there. Screenshot pasted below

I've updated this PR add an 'Identifiers' page to the Mondo User Guide -- I'm not sure what other content would make sense on that page, but if this approach makes more sense than adding this paragraph to the README, then that would be fine by me too!

Copy link
Member

@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.

Lets merge this and reconsider it during our Mondo documentation hackathon in June!

@gaurav gaurav requested a review from twhetzel May 2, 2024 15:04
@nicolevasilevsky
Copy link
Member

Lets merge this and reconsider it during our Mondo documentation hackathon in June!

when is the hackathon in June?

@matentzn
Copy link
Member

matentzn commented May 3, 2024

when is the hackathon in June?

No date yet.

@gaurav
Copy link
Contributor Author

gaurav commented May 16, 2024

Lets merge this and reconsider it during our Mondo documentation hackathon in June!

That sounds good to me, but of course I don't get a vote :)

@matentzn matentzn merged commit 24209df into monarch-initiative:master May 19, 2024
1 check passed
@matentzn
Copy link
Member

@gaurav thank you for your patience and contribution here! Really sorry it took so long! We highly appreciate any further suggestions as well :)

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.

None yet

4 participants