-
-
Notifications
You must be signed in to change notification settings - Fork 79
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
Cohort example ignores first cohort #16
Comments
It appears that the return object is leaving off the first row of the results, but it's running the whole query. So consider the following example:
This returns a 7x4 table. The August 2015 cohort is missing the 0-month row. So that's where the issue lies. |
Thanks very much for this and detective work, definitely a bug that I will look to fix ASAP. |
Well good news is that this is now fixed, bad news is that it was affecting all v4 calls, missing out the first row. This was due to batching where I added the pageToken to the call, and in my R-brain I indexed it from 1, whereas it should have been from 0. I will submit this as a patch to the CRAN version ASAP. Thanks very much for the report! |
Thank you for developing the package!
|
This is now on CRAN, version 0.1.1. |
Dear Mark,
First of all, thank you for writing this great R package! I am delighted to be able to use the API v4 directly from R.
I tried out the cohort example from the vignette, but it didn't behave the way I expected.
This only returns a value for
cohort 2
. My first thought was that we didn't have data for cohort 1, but if I swapped the date ranges, I got only data for 2015-08-01. In fact, if I added a third row, I get the last two:The above returns cohorts 2 and 3. Seems to be a simple off-by-one. I am using R 3.2.4 and I installed via
install.packages
today.The text was updated successfully, but these errors were encountered: