search

USDA-ARS-NWRC / smrf

SMRF was designed to increase the flexibility of taking measured weather data, or atmospheric models, and distributing the data across a watershed.
Other
12 stars 4 forks source link

snow.py future warning #85

Open scotthavens opened 5 years ago

scotthavens commented 5 years ago

Got a warning from snow.py about a future warning.

/home/scotthavens/Projects/wrf_paper/.venv/local/lib/python3.5/site-packages/smrf/envphys/snow.py:208: FutureWarning: Using a non-tuple sequence for multidimensional indexing is deprecated; use `arr[tuple(seq)]` instead of `arr[seq]`. In the future this will be interpreted as an array index, `arr[np.array(seq)]`, which will result either in an error or a different result.
  ps[ind] = row['snow']
/home/scotthavens/Projects/wrf_paper/.venv/local/lib/python3.5/site-packages/smrf/envphys/snow.py:211: FutureWarning: Using a non-tuple sequence for multidimensional indexing is deprecated; use `arr[tuple(seq)]` instead of `arr[seq]`. In the future this will be interpreted as an array index, `arr[np.array(seq)]`, which will result either in an error or a different result.
  sd[ind] = row['density']
micahjohnson150 commented 5 years ago

Thanks, I will take a look!

jomey commented 4 years ago

I think this is outdated, no?

scotthavens commented 4 years ago

I'm not entirely sure. I haven't seen the future warning again but these lines are still in there and might be worth ensuring that it's doing what we want. Issues like this may be why the newest numpy version changed the tests #169