Skip to content

Re-enable OWL generation - #527

Closed
cthoyt wants to merge 1 commit into
masterfrom
delete-owl
Closed

Re-enable OWL generation#527
cthoyt wants to merge 1 commit into
masterfrom
delete-owl

Conversation

@cthoyt

@cthoyt cthoyt commented Apr 4, 2026

Copy link
Copy Markdown
Member

This pull request re-enables generating owl in the make gen-project command.

It's not documented, but I assume the reason that OWL was excluded was there were upstream bugs in the LinkML implementation that were causing it to fail. I reported this as a bug in linkml/linkml#3358 and sent a pull request that fixes it in linkml/linkml#3359.

This PR includes the output of the OWL generation after applying the patch to LinkML

Note: the upstream PR was merged promptly, but tests in this PR won't pass until the upstream linkml repository gets released to PyPI containing that fix

@matentzn

matentzn commented Apr 4, 2026

Copy link
Copy Markdown
Collaborator

@cthoyt I need to do this - this is a little more complex. I will try to handle it next week.

@cthoyt

cthoyt commented Apr 4, 2026

Copy link
Copy Markdown
Member Author

@matentzn this is super low priority, it's just something I noticed was weird that wasn't changing during the re-generation

@matentzn

Copy link
Copy Markdown
Collaborator

Sorry I thought this was something else. Why do we need a OWL rendering of the SSSOM schema?

@cthoyt

cthoyt commented Apr 22, 2026

Copy link
Copy Markdown
Member Author

🤷 it seems to be something built into LinkML output. One could also ask "why do we need protobuf"

either way, there is something that's already been output before, and it's awfully out of date. so we should either update it, or delete it.

@matentzn

Copy link
Copy Markdown
Collaborator

either way, there is something that's already been output before, and it's awfully out of date. so we should either update it, or delete it.

I would prefer if we get rid of the OWL representation of the schema for now.

@cthoyt

cthoyt commented Apr 22, 2026

Copy link
Copy Markdown
Member Author

@matentzn okay, deleted this file in #539 as an alternative

@cthoyt cthoyt closed this Apr 22, 2026
@cthoyt
cthoyt deleted the delete-owl branch April 22, 2026 20:10
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.

2 participants