Closed nmdefries closed 1 year ago
Running it currently, and it may be a good idea to have it print some statements to the log as it starts, as running ./run.sh
, it's very hard to tell what's happening initially, or if it is even running at all.
So apparently I should've been looking at run.Rout
as well as nohup.out
; might want to add a print statement for that to the .sh
file.
Pretty much exactly what I was hoping this would be, thank you!
Your list of todos makes sense, and some of them are more a problem of adding some forecasters on the other end of the pipeline. Things I would add, all of which are fairly minor details that can probably wait:
Final thing to think about is the names; with my PR the names are unique, but still not very interpretable. We might want to use tar_read(forecasters)
to make saner names somehow (e.g. list the forecaster and some useful subset of the parameters). Maybe a table that lists the scores chosen and their associated parameters, and a lookup thing? Definitely a project for later
The app displays data regardless of whether the scores were made this time or not. I thought tar_objects
(to get forecaster/output names) would be tied to the current run. Maybe there's a setting to control that. We could also clear the cache every time, but then we wouldn't get speedups from targets
.
The remaining changes are:
run.R
to run the targets and then the shiny app, so that shiny inputs (target outputs) are always updated. The idea is to always useRscript run.R
to launch the app. This doesn't prevent the app from being run with old targets output, but would make it a bit inconvenient via requiring the user to manually setforecaster_options
(currently set inrun.R
, not in the app)../app.R
based off of the flu hosp exploration app. Modified this to only load one forecaster's data at a time and cache the result (up to 1GB memory use total).master
->main
switch.)Notes:
renv
updates correctly, still messing around with that.