-
Notifications
You must be signed in to change notification settings - Fork 53
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
How to link raw data to prefix maps? #667
Comments
@matentzn maybe this is a good time to deprecate |
I cant say I am not tempted but I fear that ship has sailed. This is such a fundamental part of the model that it will require a storm of people to convince me to really get rid of it. That said, introducing a new field, "external_prefix_map" would be one way to deal with this issue. To get the debate furthered - what exactly is the reason for allowing EPMs in this context? In my current view, this just complicates processing IMO. What is required here is a bimap, not an EPM. |
my alternate idea to extend what's allowed in |
The suggestion is not unreasonable, but what is the use case of allowing EPMs at all in this context? |
SSSOM allows embedding prefix maps into data to reduce the risk of uncoupled data from prefix maps. For example, we can embed prefix maps into our tsvs files like this:
In hindsight, choosing the term
curie_map
was probably not ideal,prefix_map
would have been better.In any case, maybe bioregistry should develop a recommendations for data providers to document their use of prefixes. It could be, that say, a link to a context like https://bioregistry.io/context/obo is sufficient, but it would go a long way to standardise this.
The text was updated successfully, but these errors were encountered: