PyNGL projection resources broken on Python 3.x

NCAR / wrf-python

A collection of diagnostic and interpolation routines for use with output from the Weather Research and Forecasting (WRF-ARW) Model.

https://wrf-python.readthedocs.io

Apache License 2.0

410 stars 155 forks source link

PyNGL projection resources broken on Python 3.x #57

Closed bladwig1 closed 6 years ago

bladwig1 commented 6 years ago

PyNGL for Python 3.x uses the native string type for the resources. Currently WRF-Python promotes Python 2.x strings to Unicode, then back to bytes to make the PyNGL resources. This breaks PyNGL on Python 3.x

bladwig1 commented 6 years ago

Overall, the string handling should be changed to be native to the Python version used. This may introduce some backwards compatibility issues for the xarray name field, but for the most part, it should be transparent to most users. However, it should probably be released as a major version number bump for this reason.