search

ga4gh / ga4gh-schemas

Models and APIs for Genomic data. RETIRED 2018-01-24
http://ga4gh.org
Apache License 2.0
214 stars 114 forks source link

Integrating external identifiers #807

Closed mbaudis closed 7 years ago

david4096 commented 7 years ago

Thanks @mbaudis we can definitely offer some light federation this way! Would you be so kind as to look over https://github.com/ga4gh/schemas/pull/761 ? It applies this concept across the API.

Since we have proposed adding the metadata field in the Timestamps PR, we might use it for this external_identifiers message.

mbaudis commented 7 years ago

@david4096 #761 is probably fine? However, and I actually don't want to push this, but: What is an OntologyTerm other than an external identifier? For which you need a service, if you want to get it resolved. By CURIE.

Just saying: ExternalIdentifier & OntologyTerm seem basically the same - or?

david4096 commented 7 years ago

@mbaudis Yep! However, I don't expect all these other databases to present themselves as ontologies. For that reason we give a message that better suits the use case. Does that make sense?

mbaudis commented 7 years ago

Merging into the metadata integration branch