-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
xarray to and from iris #814
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 5 commits
f7842b3
e0498c5
c261845
edae053
cd92bca
44930af
6bed306
cd06a2e
15fbbba
e7f9cb1
03e9076
877d06f
f48de5a
e42aeb2
338ef6b
46f68ff
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,24 +3,40 @@ | |
import numpy as np | ||
|
||
from .core.dataarray import DataArray | ||
from .core.pycompat import OrderedDict | ||
from .conventions import ( | ||
maybe_encode_timedelta, maybe_encode_datetime, decode_cf) | ||
|
||
ignored_attrs = set(['name', 'tileIndex']) | ||
cdms2_ignored_attrs = set(['name', 'tileIndex']) | ||
iris_forbidden_keys = set( | ||
['standard_name', 'long_name', 'units', 'bounds', 'axis', | ||
'calendar', 'leap_month', 'leap_year', 'month_lengths', | ||
'coordinates', 'grid_mapping', 'climatology', | ||
'cell_methods', 'formula_terms', 'compress', | ||
'missing_value', 'add_offset', 'scale_factor', | ||
'valid_max', 'valid_min', 'valid_range', '_FillValue']) | ||
cell_methods_strings = set(['point', 'sum', 'maximum', 'median', 'mid_range', | ||
'minimum', 'mean', 'mode', 'standard_deviation', | ||
'variance']) | ||
|
||
|
||
def encode(var): | ||
return maybe_encode_timedelta(maybe_encode_datetime(var.variable)) | ||
|
||
|
||
def filter_attrs(_attrs, ignored_attrs): | ||
return dict((k, v) for k, v in _attrs.items() if k not in ignored_attrs) | ||
|
||
|
||
def from_cdms2(variable): | ||
"""Convert a cdms2 variable into an DataArray | ||
""" | ||
def get_cdms2_attrs(var): | ||
return dict((k, v) for k, v in var.attributes.items() | ||
if k not in ignored_attrs) | ||
|
||
values = np.asarray(variable) | ||
name = variable.id | ||
coords = [(v.id, np.asarray(v), get_cdms2_attrs(v)) | ||
coords = [(v.id, np.asarray(v), | ||
filter_attrs(v.attributes, cdms2_ignored_attrs)) | ||
for v in variable.getAxisList()] | ||
attrs = get_cdms2_attrs(variable) | ||
attrs = filter_attrs(variable.attributes, cdms2_ignored_attrs) | ||
dataarray = DataArray(values, coords=coords, name=name, attrs=attrs) | ||
return decode_cf(dataarray.to_dataset())[dataarray.name] | ||
|
||
|
@@ -31,12 +47,9 @@ def to_cdms2(dataarray): | |
# we don't want cdms2 to be a hard dependency | ||
import cdms2 | ||
|
||
def encode(var): | ||
return maybe_encode_timedelta(maybe_encode_datetime(var.variable)) | ||
|
||
def set_cdms2_attrs(var, attrs): | ||
def set_cdms2_attrs(_var, attrs): | ||
for k, v in attrs.items(): | ||
setattr(var, k, v) | ||
setattr(_var, k, v) | ||
|
||
axes = [] | ||
for dim in dataarray.dims: | ||
|
@@ -49,3 +62,142 @@ def set_cdms2_attrs(var, attrs): | |
cdms2_var = cdms2.createVariable(var.values, axes=axes, id=dataarray.name) | ||
set_cdms2_attrs(cdms2_var, var.attrs) | ||
return cdms2_var | ||
|
||
|
||
# TODO: Add converting bounds from xarray to Iris and back | ||
def to_iris(dataarray): | ||
"""Convert a DataArray into a Iris Cube | ||
""" | ||
# Iris not a hard dependency | ||
import iris | ||
# iris.unit is deprecated in Iris v1.9 | ||
import cf_units | ||
|
||
def check_attrs(attrs, keys): | ||
return dict((k, v) for k, v in attrs.items() if k in keys) | ||
|
||
def get_args(attrs): | ||
_args = {'attributes': filter_attrs(attrs, iris_forbidden_keys)} | ||
|
||
_args.update(check_attrs(attrs, ('standard_name', 'long_name',))) | ||
_unit_args = check_attrs(coord.attrs, ('calendar',)) | ||
if 'units' in attrs: | ||
_args['units'] = cf_units.Unit(attrs['units'], **_unit_args) | ||
return _args | ||
|
||
def get_cell_methods(cell_methods_str): | ||
"""Converts string to iris cell method objects""" | ||
cell_methods = [] | ||
_cell_method_words = [w.strip() for w in cell_methods_str.split(':')] | ||
cm = {'coords': [], 'method': '', 'interval': [], 'comment': []} | ||
skip = False | ||
for i, word in enumerate(_cell_method_words): | ||
# If this value is a comment or an interval don't read | ||
if skip: | ||
skip = False | ||
continue | ||
# If this word is an axis | ||
if word not in cell_methods_strings | set(['interval', 'comment']): | ||
|
||
# If we already have a method this must be the next cell_method | ||
if cm['method']: | ||
cell_methods.append( | ||
iris.coords.CellMethod(cm['method'], | ||
coords=cm['coords'], | ||
intervals=cm['interval'], | ||
comments=cm['comment'])) | ||
cm = {'coords': [], 'method': '', 'interval': [], | ||
'comment': []} | ||
cm['coords'].append(word) | ||
continue | ||
else: | ||
cm['coords'].append(word) | ||
elif word in ['interval', 'comment']: | ||
cm[word].append(_cell_method_words[i + 1]) | ||
skip = True | ||
continue | ||
else: | ||
cm['method'] = word | ||
else: | ||
cell_methods.append( | ||
iris.coords.CellMethod(cm['method'], coords=cm['coords'], | ||
intervals=cm['interval'], | ||
comments=cm['comment'])) | ||
return cell_methods | ||
|
||
dim_coords = [] | ||
aux_coords = [] | ||
|
||
for coord_name in dataarray.coords: | ||
coord = encode(dataarray.coords[coord_name]) | ||
coord_args = get_args(coord.attrs) | ||
coord_args['var_name'] = coord_name | ||
axis = None | ||
if coord.dims: | ||
axis = dataarray.get_axis_num(coord.dims) | ||
if coord_name in dataarray.dims: | ||
iris_coord = iris.coords.DimCoord(coord.values, **coord_args) | ||
dim_coords.append((iris_coord, axis)) | ||
else: | ||
iris_coord = iris.coords.AuxCoord(coord.values, **coord_args) | ||
aux_coords.append((iris_coord, axis)) | ||
|
||
args = get_args(dataarray.attrs) | ||
args['var_name'] = dataarray.name | ||
args['dim_coords_and_dims'] = dim_coords | ||
args['aux_coords_and_dims'] = aux_coords | ||
if 'cell_methods' in dataarray.attrs: | ||
args['cell_methods'] = get_cell_methods(dataarray.attrs['cell_methods']) | ||
|
||
cube = iris.cube.Cube(dataarray.to_masked_array(), **args) | ||
return cube | ||
|
||
|
||
def from_iris(cube): | ||
"""Convert a Iris cube into an DataArray | ||
""" | ||
|
||
def get_attr(_obj): | ||
attrs = {'standard_name': _obj.standard_name, | ||
'long_name': _obj.long_name} | ||
if _obj.units.calendar: | ||
attrs['calendar'] = _obj.units.calendar | ||
if _obj.units.origin != '1': | ||
attrs['units'] = _obj.units.origin | ||
attrs.update(_obj.attributes) | ||
return dict((k, v) for k, v in attrs.items() if v is not None) | ||
|
||
def get_cell_methods(cell_methods_obj): | ||
_cell_methods = [] | ||
for cell_method in cell_methods_obj: | ||
names = ''.join(['{}: '.format(n) for n in cell_method.coord_names]) | ||
intervals = ' '.join(['interval: {}'.format(interval) | ||
for interval in cell_method.intervals]) | ||
comments = ' '.join(['comment: {}'.format(comment) | ||
for comment in cell_method.comments]) | ||
extra = ' '.join([intervals, comments]).strip() | ||
if extra: | ||
extra += ' ' | ||
_cell_methods.append(names + cell_method.method + extra) | ||
return ' '.join(_cell_methods) | ||
|
||
name = cube.var_name | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is it possible for a cube not to have variable name? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think this could possible be None. It's the CF variable name but I think the cube could have been made with out reading in a file so that the Name would be None. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. A quick search in the Iris source gives this: @property
def var_name(self):
"""The CF variable name for the object."""
return self._var_name
@var_name.setter
def var_name(self, name):
if name is not None:
if not name:
raise ValueError('An empty string is not a valid CF variable '
'name.')
elif set(name).intersection(string.whitespace):
raise ValueError('{!r} is not a valid CF variable name because'
' it contains whitespace.'.format(name))
self._var_name = name So would conclude that's it's not an empty string or None |
||
dims = [dim.var_name for dim in cube.dim_coords] | ||
if not dims: | ||
|
||
dims = ["dim{}".format(i) for i in range(cube.data.ndim)] | ||
coords = OrderedDict() | ||
|
||
for coord in cube.coords(): | ||
coord_attrs = get_attr(coord) | ||
coord_dims = [dims[i] for i in cube.coord_dims(coord)] | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is it possible to have coord_dims(coord) return a tuple that does not have length equal to it's number of dimensions? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. coord_dims(coord):
So I think it's ok. |
||
if coord_dims: | ||
coords[coord.var_name] = (coord_dims, coord.points, coord_attrs) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it's also possible to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is there an xarray exception that might make sense for the error? Or just There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We usually just raise ValueError for cases like this where we cannot handle On Wed, Aug 10, 2016 at 2:01 PM, Neil Parley [email protected]
|
||
else: | ||
coords[coord.var_name] = ((), | ||
np.asscalar(coord.points), coord_attrs) | ||
|
||
array_attrs = get_attr(cube) | ||
cell_methods = get_cell_methods(cube.cell_methods) | ||
if cell_methods: | ||
array_attrs['cell_methods'] = cell_methods | ||
dataarray = DataArray(cube.data, coords=coords, name=name, | ||
attrs=array_attrs, dims=dims) | ||
return decode_cf(dataarray.to_dataset())[dataarray.name] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,6 +8,7 @@ | |
Coordinate, Variable) | ||
from xarray.core.pycompat import iteritems, OrderedDict | ||
from xarray.core.common import _full_like | ||
from xarray.conventions import maybe_encode_datetime | ||
from . import (TestCase, ReturnItem, source_ndarray, unittest, requires_dask, | ||
requires_bottleneck) | ||
|
||
|
@@ -1621,15 +1622,15 @@ def test_to_and_from_cdms2(self): | |
[('distance', [-2, 2], {'units': 'meters'}), | ||
('time', pd.date_range('2000-01-01', periods=3))], | ||
name='foo', attrs={'baz': 123}) | ||
expected_coords = [Coordinate('distance', [-2, 2]), | ||
Coordinate('time', [0, 1, 2])] | ||
|
||
actual = original.to_cdms2() | ||
self.assertArrayEqual(actual, original) | ||
self.assertEqual(actual.id, original.name) | ||
self.assertItemsEqual(actual.getAxisIds(), original.dims) | ||
for axis, coord in zip(actual.getAxisList(), expected_coords): | ||
for axis, coord_key in zip(actual.getAxisList(), original.coords): | ||
coord = original.coords[coord_key] | ||
self.assertEqual(axis.id, coord.name) | ||
self.assertArrayEqual(axis, coord.values) | ||
self.assertArrayEqual(axis, maybe_encode_datetime(coord).values) | ||
self.assertEqual(actual.baz, original.attrs['baz']) | ||
|
||
component_times = actual.getAxis(1).asComponentTime() | ||
|
@@ -1639,6 +1640,54 @@ def test_to_and_from_cdms2(self): | |
roundtripped = DataArray.from_cdms2(actual) | ||
self.assertDataArrayIdentical(original, roundtripped) | ||
|
||
def test_to_and_from_iris(self): | ||
try: | ||
import iris | ||
import cf_units | ||
except ImportError: | ||
raise unittest.SkipTest('iris not installed') | ||
|
||
|
||
coord_dict = OrderedDict() | ||
coord_dict['distance'] = ('distance', [-2, 2], {'units': 'meters'}) | ||
coord_dict['time'] = ('time', pd.date_range('2000-01-01', periods=3)) | ||
coord_dict['height'] = 10 | ||
coord_dict['distance2'] = ('distance', [0, 1], {'foo': 'bar'}) | ||
coord_dict['time2'] = (('distance', 'time'), [[0, 1, 2], [2, 3, 4]]) | ||
|
||
original = DataArray(np.arange(6).reshape(2, 3), coord_dict, | ||
name='Temperature', attrs={'baz': 123, | ||
'units': 'Kelvin', | ||
'standard_name': | ||
'fire_temperature', | ||
'long_name': | ||
'Fire Temperature'}, | ||
dims=('distance', 'time')) | ||
|
||
actual = original.to_iris() | ||
self.assertArrayEqual(actual.data, original.data) | ||
self.assertEqual(actual.var_name, original.name) | ||
self.assertItemsEqual([d.var_name for d in actual.dim_coords], | ||
original.dims) | ||
|
||
for coord, orginal_key in zip((actual.coords()), original.coords): | ||
original_coord = original.coords[orginal_key] | ||
self.assertEqual(coord.var_name, original_coord.name) | ||
self.assertArrayEqual(coord.points, | ||
maybe_encode_datetime(original_coord).values) | ||
self.assertEqual(actual.coord_dims(coord), | ||
original.get_axis_num | ||
(original.coords[coord.var_name].dims)) | ||
|
||
|
||
self.assertEqual(actual.coord('distance2').attributes['foo'], | ||
original.coords['distance2'].attrs['foo']) | ||
self.assertEqual(actual.coord('distance').units, | ||
cf_units.Unit(original.coords['distance'].units)) | ||
self.assertEqual(actual.attributes['baz'], original.attrs['baz']) | ||
self.assertEqual(actual.standard_name, original.attrs['standard_name']) | ||
|
||
roundtripped = DataArray.from_iris(actual) | ||
self.assertDataArrayIdentical(original, roundtripped) | ||
|
||
def test_to_dataset_whole(self): | ||
unnamed = DataArray([1, 2], dims='x') | ||
with self.assertRaisesRegexp(ValueError, 'unable to convert unnamed'): | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there a reason you needed to change this variable name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like variable shadowing so I prefer to add the underscore, it's because I have had to deal with some really badly written academic code. Here it's clear enough so can change it back.