|
115 | 115 | # Type for Mapper functions
|
116 | 116 | Mapper = Callable[
|
117 | 117 | [Union[xr.DataArray, xr.Dataset], str, bool, str],
|
118 |
| - Union[Optional[str], List[Optional[str]], DataArray], # this sucks |
| 118 | + Union[List[Optional[str]], DataArray], # this sucks |
119 | 119 | ]
|
120 | 120 |
|
121 | 121 |
|
@@ -372,7 +372,7 @@ def _getattr(
|
372 | 372 | newmap = dict()
|
373 | 373 | unused_keys = set(attribute.keys())
|
374 | 374 | for key in _AXIS_NAMES + _COORD_NAMES:
|
375 |
| - value = _get_axis_coord(obj, key, error=False, default=None) |
| 375 | + value = _get_axis_coord(obj, key, error=False) |
376 | 376 | unused_keys -= set(value)
|
377 | 377 | if value != [None]:
|
378 | 378 | good_values = set(value) & set(obj.dims)
|
@@ -694,13 +694,13 @@ def get_valid_keys(self) -> Set[str]:
|
694 | 694 | varnames = [
|
695 | 695 | key
|
696 | 696 | for key in _AXIS_NAMES + _COORD_NAMES
|
697 |
| - if _get_axis_coord(self._obj, key, error=False, default=None) != [None] |
| 697 | + if _get_axis_coord(self._obj, key, error=False) != [None] |
698 | 698 | ]
|
699 | 699 | with suppress(NotImplementedError):
|
700 | 700 | measures = [
|
701 | 701 | key
|
702 | 702 | for key in _CELL_MEASURES
|
703 |
| - if _get_measure(self._obj, key, error=False, default=None) != [None] |
| 703 | + if _get_measure(self._obj, key, error=False) != [None] |
704 | 704 | ]
|
705 | 705 | if measures:
|
706 | 706 | varnames.extend(measures)
|
|
0 commit comments