openradar / TINT

TINT Is Not TITAN. Python code for tracking objects. Specifically storm cells.
BSD 2-Clause "Simplified" License
80 stars 43 forks source link

suggested_corrections #47

Open CloeDavid opened 9 months ago

CloeDavid commented 9 months ago

I noticed some warnings and errors when started using TINT. I'm working with Python 3.9 and I updated recently most of the common packages (numpy, pandas, xarray, etc)

I'd like to suggest some updates.

CloeDavid commented 9 months ago

https://github.com/openradar/TINT/blob/49dddec376c6d5bb6da771f5f0d1bdab1205e8be/tint/grid_utils.py#L46-L48

In line 48 np.int( ) is now deprecated and needs to be simply replaced by int( )

CloeDavid commented 9 months ago

https://github.com/openradar/TINT/blob/49dddec376c6d5bb6da771f5f0d1bdab1205e8be/tint/data_utils.py#L10

boto module is not in the required dependencies (user have to download it after an error message). Also boto is no more supported as boto3 has replaced it.

zssherman commented 9 months ago

@CloeDavid Thanks for the heads up! Yeah development has slowed here as TINT is currently being integrated into tobac https://github.com/tobac-project/tobac/tree/v2.0-dev version 2.0. I'll make the changes now to fix those two depreciations for now. We do recommend moving towards tobac as development is mostly focused there now. Thanks again for the heads up!