Skip to content

Commit ee0bdaa

Browse files
Apply ruff/flake8-simplify rule SIM910
SIM910 Use `.get(...)` instead of `.get(..., None)`
1 parent 98ceaa6 commit ee0bdaa

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

xarray/core/indexes.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1247,7 +1247,7 @@ def create_variables(
12471247
level = name
12481248
dtype = self.level_coords_dtype[name] # type: ignore[index] # TODO: are Hashables ok?
12491249

1250-
var = variables.get(name, None)
1250+
var = variables.get(name)
12511251
if var is not None:
12521252
attrs = var.attrs
12531253
encoding = var.encoding

xarray/structure/merge.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,7 @@ def merge_collected(
300300
variables = [variable for variable, _ in elements_list]
301301
try:
302302
merged_vars[name] = unique_variable(
303-
name, variables, compat, equals.get(name, None)
303+
name, variables, compat, equals.get(name)
304304
)
305305
except MergeError:
306306
if compat != "minimal":

xarray/tests/test_units.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -234,9 +234,7 @@ def convert_units(obj, to):
234234
elif isinstance(obj, xr.DataArray):
235235
name = obj.name
236236

237-
new_units = (
238-
to.get(name, None) or to.get("data", None) or to.get(None, None) or None
239-
)
237+
new_units = to.get(name) or to.get("data") or to.get(None) or None
240238
data = convert_units(obj.variable, {None: new_units})
241239

242240
coords = {

0 commit comments

Comments
 (0)