-
Notifications
You must be signed in to change notification settings - Fork 17
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
Memory leak #91
Comments
Hi,
The only thing that came up was Attached rds if you want to take a look.
|
Hi John, Thanks for taking a look (and for the offer of help - see email). I haven't tried lineprof (only profvis) so I will take a look. Even a small memory leak here could be at fault if it scales with samples (1000 per country so ~ 1Gb per country at least). Sam |
Further investigation indicates that the root cause of the memory leak is the Dropping the
Appears to fix the RAM issue. This is not a drop-in replacement however as in testing the |
When running over many regions there is a memory leak that causes an out of RAM error and ultimately leading to estimation failing.
There is a gist here that provides a reprex.
Via turning areas in the code off I believe I have localized the issue to here:
EpiNow/R/estimate_R0.R
Lines 190 to 239 in 0d69358
This area of the code runs a forecast and relies on
EpiSoon
,data.table
,purrr
,fable
andfabletools
any of which may be the source of the problem.This deduction may also be a false positive with another area of the code to blame. All code here that is not from other packages has recently been rewritten from the
tidyverse
todata.table
.The text was updated successfully, but these errors were encountered: