cesium-ml / cesium

Machine Learning Time-Series Platform
Other
670 stars 101 forks source link

Reorganize `time_series.from_netcdf` to only open file once #184

Closed bnaul closed 8 years ago

bnaul commented 8 years ago

Hopefully fixes issue when loading time series on dask.distributed workers.

stefanv commented 8 years ago

🐽

Why are all those binary files changed, btw?

bnaul commented 8 years ago

Those are test data; I renamed some attribute based on what I mentioned earlier (name being protected) and those files needed to be changed accordingly.

stefanv commented 8 years ago

Ok, looks good to me.

acrellin commented 8 years ago

Looks good