Skip to content
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

Request dplyr package 1.0.0 for PH290W: R for Public Health #1810

Closed
lytello opened this issue Sep 8, 2020 · 8 comments · Fixed by #1824
Closed

Request dplyr package 1.0.0 for PH290W: R for Public Health #1810

lytello opened this issue Sep 8, 2020 · 8 comments · Fixed by #1824
Assignees
Labels

Comments

@lytello
Copy link

lytello commented Sep 8, 2020

dplyr 1.0.0

r.datahub

PHW290W: R for Public Health

https://classes.berkeley.edu/content/2020-fall-pbhlth-290-009-sem-009

Instructor: Will Wheeler, Lawrence Tello

Fall 2020

@lytello lytello added the support label Sep 8, 2020
@felder
Copy link
Contributor

felder commented Sep 9, 2020

I asked @lytello in slack to confirm that this won't interfere with PH250F/G and update the issue here. Once done, I'll attempt to deploy.

@felder felder self-assigned this Sep 9, 2020
@lytello
Copy link
Author

lytello commented Sep 9, 2020

Confirmed: PH250F/G is not using R as they removed all R components of the course.

@felder
Copy link
Contributor

felder commented Sep 9, 2020

@lytello ok, I'll get to work on it. You'll see updates in this issue as I make commits.

@felder
Copy link
Contributor

felder commented Sep 9, 2020

@lytello I'm upgrading dplyr to 1.0.2, I hope that's ok. Please let me know if it is not.

@felder
Copy link
Contributor

felder commented Sep 9, 2020

Running into the github api limit. I'll try later.

@yuvipanda does this issue I filed for hubploy (berkeley-dsep-infra/hubploy#42) seem at all feasible? Any other ideas for getting a github_pat into the build environment?

@felder
Copy link
Contributor

felder commented Sep 10, 2020

After waiting for the API limits to refresh, it still no worky. There's just too many calls to github with the addition of the new dependencies.

I'm talking with @ryanlovett to try and see if we can work around this.

@yuvipanda
Copy link
Contributor

I bumped up R to 4.0 (#1824), and with it got new dplyr. I was able to work-around the issues we ran into when we last tried to do this in #1786.

@lytello
Copy link
Author

lytello commented Sep 11, 2020

Thank you for the quick turn around!

@felder felder closed this as completed Sep 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants