Skip to content

Update renv to reduce GH package use#1013

Open
jashapiro wants to merge 5 commits into
masterfrom
jashapiro/patch-rstudio
Open

Update renv to reduce GH package use#1013
jashapiro wants to merge 5 commits into
masterfrom
jashapiro/patch-rstudio

Conversation

@jashapiro

Copy link
Copy Markdown
Member

To get around errors we had been running into with RStudio server when some process was causing 100% cpu usage at random times, I am patching a line from the internal RStudio file SessionEnvironment.R, changing the same code that has previously been implicated in similar bugs. (See https://github.com/AlexsLemonade/rstudio-server/issues/184, as well as rstudio/rstudio#13489 and https://github.com/rstudio/rstudio/pull/13529/changes

Along the way, I am also updating the renv.lock file here to use the latest version of renv as well as Bioconductor release of Banksy and the CRAN version of stringfish, which were previously set to GitHub versions.

@jashapiro

jashapiro commented Jun 5, 2026

Copy link
Copy Markdown
Member Author

Hmmm... everything worked but qs

I suspect that might be it for qs. Our best option may be to convert the files and keep .rds versions of them on S3.

Update: converted qs to install from the archived version on RSPM, which may did help

@jashapiro jashapiro changed the title Try patching RStudio Update renv.lock Jun 8, 2026
@jashapiro jashapiro changed the title Update renv.lock Update renv to reduce GH package use Jun 8, 2026
@jashapiro

Copy link
Copy Markdown
Member Author

Given that the RStudio patch doesn't seem to work, I reverted that change, and this PR is now just about standardizing the renv package versions. Assuming it builds, it should be ready for review.

@jashapiro jashapiro marked this pull request as ready for review June 17, 2026 14:01
@jashapiro jashapiro requested a review from sjspielman June 17, 2026 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant