cmu-delphi / forecast-eval

delphi.cmu.edu/forecast-eval
MIT License
5 stars 2 forks source link

Remove, revise, or replace `start_dev` target #283

Open nmdefries opened 1 year ago

nmdefries commented 1 year ago

The start_dev target is not useful. It runs bash inside a docker container, whic his easy enough to do by hand. It's also unclear which docker container it's actually trying to use.

The target runs the ghcr.io/cmu-delphi/forecast-eval image, which is meant for the dashboard, but mounts the Report and dist dirs that are only needed to run the pipeline, not the dashboard.

Three options here: