Bioconductor / GenomicDataCommons

Provide R access to the NCI Genomic Data Commons portal.
http://bioconductor.github.io/GenomicDataCommons/
83 stars 23 forks source link

Need to return more complex object to support counting hits and facets #26

Closed seandavi closed 7 years ago

seandavi commented 7 years ago

See #16 for details on facets and the need for this proposed change.

Currently, the API calls return only the "hits" data. To support counting of results (in the pagination JSON object) and facets (in the aggregation JSON object), we need to return a more complicated object. I am proposing that we return a list with three elements:

seandavi commented 7 years ago

Also, see #15.

lawremi commented 7 years ago

Assuming this is happening at a fairly low level, probably should at least slap a class on the list for integrity reasons. See ListSolrResult.

seandavi commented 7 years ago

That is the thinking. Thanks.

On Jan 7, 2017 2:10 PM, "Michael Lawrence" notifications@github.com wrote:

Assuming this is happening at a fairly low level, probably should at least slap a class on the list for integrity reasons. See ListSolrResult.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/Bioconductor/GenomicDataCommons/issues/26#issuecomment-271103438, or mute the thread https://github.com/notifications/unsubscribe-auth/AAFpEyJhiD5yUZ2sY0R5GTxBhHYMlL_xks5rP-M9gaJpZM4LdfDv .

seandavi commented 7 years ago

first-pass at complex object complete. See GDCResponse. Incorporated into tag 0.4.0.