-
Notifications
You must be signed in to change notification settings - Fork 23
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
improve performance #17
Comments
No worries Nicole, it's a team effort - that's what makes it fun :) |
Going forward, performant access to the API is predicated by using the RPC interface instead of the REST interface: I am working on writing a running example based on gRPC. |
Does this mean that Avro is getting canned? And an R gRPC client coming? AWESO^H^H^H^H^Hthat is an eminently logical implementation decision. Statistics is the grammar of science. On Mon, Jul 6, 2015 at 4:31 PM, Siddhartha Bagaria <notifications@github.com
|
I am trying to base the gRPC R package on the RProtoBuf package. But there are several changes needed there too. A generic package might be a little farther off but I can get the GoogleGenomics endpoints running soon-ish. |
Sid, how about just basically writing a wrapper via Rcpp? Autonomy is always more fun :) Tim, I think it's an evolution :) |
Paul, that is what will happen if I base it on the RProtoBuf package. I need the package for the interface to the C protocol buffers library. |
Gotcha! Super-excited to see what you come up with :) Thx, |
Resolved with #56 |
This package is in active development and there are some inefficiencies in a few places (e.g., the code that accumulates multiple pages of variants and reads). But we'll tackle improving performance a bit when the code settles down a bit more.
The text was updated successfully, but these errors were encountered: