-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Description
We should DRYify get_map_data() and get_forecast_data(). There is a lot of repeated code between the two. In fact, I believe the get_map_data() table may just be the get_forecast_data() table filtered to the Hub ensemble.
Proposal
Split table creation into
- a function to generate the table (as a tibble) for all or a subset of models. Call this function
summarize_ref_date_forecasts(). - a light wrapper that generates a table using
summarize_ref_date_forecasts()and then saves it to disk at a given path. Call thiswrite_ref_date_summary(). - Two convenience functions named
write_ref_date_summary_ensemble()andwrite_ref_date_summary_all()that replicate the current functionality ofget_map_data()andget_forecast_data(), respectively
@O957 does this make sense to you?
O957
Metadata
Metadata
Assignees
Labels
No labels