-
Notifications
You must be signed in to change notification settings - Fork 1
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
GEL test bug fixes #9
Conversation
\item{page_number}{Number of page. (Optional) Default - 0} | ||
|
||
\item{page_size}{Number of entries in a page. (Optional) Default - 10} | ||
\item{size}{Number of cohort entries from database. (Optional) Default - 10} | ||
} | ||
\value{ | ||
A data frame with available cohorts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Below the png files (sorry for misplacing, cannot comment if binary 😄 )
are they escaping from packagedown building and need to be gitignored or we are good and keeping them for reference in the repo?
eg man/figures/README-unnamed-chunk-10-1.png
Either is fine, only checking if intended.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can keep them. Majority of pkgdown rendered things goes to gh-pages branch. These are used as cache.
ATM these are not big files, But on later point we can remove them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 👍
Only added 3 notes for the ToDo and a refactoring opportunity. Feel free to squash once you \
- check comments
- open issues for 3 bits mentioned above
Thank you @cgpu 🙌 :) |
* base url to api server * plot color aligning with platform * readme updates with new changes * always return defualt columns untill and end point is ready from backend * refactor the dataframe creation * Unit test data modification * minor fix * updates to cohort list function * handeling null description * minor fix * value filter josn create fix * manuall josn convert with auto_unbox = F * minor fix * unit test fixes
Improvements -
cb_list_cohort()
for user to get the complete list along side of default 10Bug found during testing with GEL data -
NULL
description in cohort incb_list_cohort()