Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

resolve regression in dependencies performance #2000

Merged
merged 10 commits into from
Sep 30, 2024

Conversation

kevinushey
Copy link
Collaborator

Addresses #1999.

@kevinushey kevinushey force-pushed the bugfix/dependencies-performance branch from 4ae7c8e to 20c380f Compare September 27, 2024 23:33
@kevinushey
Copy link
Collaborator Author

Some ad-hoc comparisons running in renv:

# renv 1.0.7
> system.time(renv::dependencies(quiet = TRUE))
   user  system elapsed 
  2.220   0.040   2.293 

# renv 1.0.8
> system.time(renv::dependencies(quiet = TRUE))
   user  system elapsed 
  8.161   0.172   6.968 

# dev
> system.time(renv::dependencies(quiet = TRUE))
   user  system elapsed 
  3.779   0.151   2.511 

@kevinushey kevinushey merged commit 16c05d5 into main Sep 30, 2024
6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant