ga4gh-beacon / beacon-v2-Models

Models that leverage the Beacon Framework v2
Apache License 2.0
4 stars 7 forks source link

Adding endpoints btwn entry types #22

Closed jrambla closed 3 years ago

jrambla commented 3 years ago

In order to make navigation easier, I've added additional endpoints with relationships between the entities. For example: so far we have individuals/{id}/biosamples and individuals/{id}/g_variants , to get the analysis performed over an individual, we need to get the biosamples, and from them, the analyses. I'm adding individuals/{id}/analyses, thus we can skip additional round trips to the server. As implementing the all endpoints is not mandatory, those Beacon instances not having such data or not having the endpoint ready could simply not implement it, BUT, the Model could be more clear and intuitive that way.

jrambla commented 3 years ago

@Tom-Shorter you are right, freedom was part of the solution, but guidance has been suggested for most aspects, filtering terms included. Guidance to strongly-enough suggest what is desired to be implemented, specially in network contexts. Thus, the question was more like "should we add these endpoints to the suggested Model?". I guess you have already answered with a 'yes' ;-)