Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you will need to talk to @ddooley:
To make FOODON URLs (http://purl.obolibrary.org/obo/foodon/releases/2025-12-30/foodon.owl) resolvable you will have to:
v2025-12-30, NOT2025-12-30.Else this intervention here wont work. Its still technically correct of course, but..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ha... I only looked at the 2025-07-31 release initially which has the
v(but the wrong release date), so I assumed the others did too. 😅There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But if tag is say "v2025-12-30" - which I'm now trying on "http://purl.obolibrary.org/obo/foodon/releases/v2025-07-31/foodon.owl" , then the above replacement URL should NOT have a "v" at end right? So tag just gets merged in to create: http://purl.obolibrary.org/obo/foodon/releases/v2025-07-31/foodon.owl . At moment prefix yeilds "http://purl.obolibrary.org/obo/foodon/releases/vv2025-07-31/foodon.owl"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or is it that a release URL, from OBO Foundry perspective just mentions date, i.e. releases/2025-07-31/foodon.owl, even though tag over on Github side always has form of e.g. https://github.com/FoodOntology/foodon/releases/tag/v2025-07-31 ????
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is correct.
You can find the expected pattern for versionIRIs inside ontology files at https://obofoundry.org/principles/fp-004-versioning.html#implementation. The extra
vin the GitHub release tag, which is NOT in the ontology versionIRI, is handled by theprefix/replacementpattern in foodon's config file here, which is what I added.Nico stated that on GitHub (not in the ontology file):
I don't know why that is. Technically, I think you could get by without the
vin GitHub releases by simply removing thevfrom the replacement part of the PURL redirectprefixrule I added, but apparently you're not supposed to do that.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@allenbaron now that you are calling me out on this, I tried to remember how this habit emerged but I simply cant!
https://chatgpt.com/share/6982fdd6-7aa0-8004-b12a-036bcb0150cf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just convention. OBO PURLs have always used dates (no
v) while GitHub projects have used thevto tag releases (usuallyv1.2.3). When we started pointing PURLs to GitHub releases, this is where we ended up.Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't mean to call anyone out. I always assumed it was inherited from software engineering, as James suggested. I do think ChatGPT makes some good points for including the
v.It seems FOODON hasn't really had the
vuntil recently, which makes me wonder if it should be required or not. I opened a new issue on the OBO Foundry repo regarding requirements/principles/standards for GitHub repos (OBOFoundry/OBOFoundry.github.io#2840).