search

SpatioTemporal / STAREPandas

STAREpandas adds SpatioTemporal Adaptive Resolution Encoding (STARE) support to pandas DataFrames. https://starepandas.readthedocs.io/en/latest/
MIT License
4 stars 1 forks source link

starelpandas.io.s3.nc4_dataset_wrapper makes end users refer to s3 -- should be transparent #135

Open michaelleerilee opened 1 year ago

michaelleerilee commented 1 year ago

End users shouldn't have to refer to a function in the s3 io module just to load a local file.

The nc4_dataset_wrapper and similar functions should perhaps be moved to the io module if possible (as a generic function).

io.s3 should contain functions depending on s3. Perhaps refactor nc4_dataset_wrapper to call a function in io.s3 that handles the s3 case.

Re: @TonHai1111