DataAssimBench
DataAssimBench copied to clipboard
RC Model fixes (and some DA fixes)
Solves #51
- Removed predict() method from reservoir computing model to avoid confusion with forecast()
- Renamed generate() to readin() to avoid confusion with data generators. This method applies Win to an input signal time series to produce reservoir state time series. Open to other names, let me know what you think.
- Related to rc model DA, updated ETKF and Var3D da cyclers to match input xarray variable names when producing analysis. Previously they always returned an xarray dataset with a data variable "x", but for the rc model the data_variable is named "r". Currently, there is no way of producing an analysis with multiple variable names (e.g. "wind" and "pressure"). These variables would have to be flattened into a single state vector before running DA. If this is a problem, we should add a github issue to this effect.