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 alternative to rdfs:label for giving python names #21

Open
rpgoldman opened this issue Jun 30, 2021 · 1 comment
Open

Add alternative to rdfs:label for giving python names #21

rpgoldman opened this issue Jun 30, 2021 · 1 comment
Milestone

Comments

@rpgoldman
Copy link

I'm not really a fan of us hijacking the rdfs:label property -- which is meant to be a human-readable name -- for the purposes of making a python-name.

Thinking of how we might do this without disturbing existing code, how about adding an annotation property like sbol:pythonIdentifier and having the sbol_factory first look for a value of pythonIdentifier and falling back to rdfs:label if it can't find a pythonIdentifier?

I was just worried that existing tools might want to use rdfs:label for their purposes and wouldn't be expecting it to be used in the way we are using it.

@rpgoldman
Copy link
Author

Jake points out, apropos of my backwards compatible suggestion that

We're also at an alpha state without many copies in the wild, and thus don't need to worry too much about backwards compatibility.

So we could actually replace the use of rdfs:label instead of shadowing it. That would be up to you all.

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

No branches or pull requests

2 participants