MazamaScience / AirFireWRF

Utilities for working with WRF atmospheric model data
https://mazamascience.github.io/AirFireWRF/
0 stars 1 forks source link

Reconcile wrf_load() and wrf_loadTimeSeries() #29

Closed tabrasel closed 3 years ago

tabrasel commented 4 years ago

Maybe we could combine the two functions into wrf_load() and have it return a list of RasterBricks. Each brick would represent a forecast hour, and each RasterLayer in a brick would store a variable. Or, if we think it's better to keep them separate, wrf_load() should at least have a more descriptive name.

One downside of the list approach is that if a user asks for just one variable over multiple hours, then a list of single-layered RasterBricks will be returned which is a bit space inefficient.

jonathancallahan commented 3 years ago

Downloading enough data to create a timeseries is not feasible so all timeseries functionality is on permanent hold.