chore: ruff rewrite dicts (#2183)

Co-authored-by: Jim Pivarski <[email protected]>
scikit-hep · Jan 31, 2023 · ac5c958 · ac5c958
1 parent cda7688
commit ac5c958
Show file tree

Hide file tree

Showing 97 changed files with 586 additions and 570 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -263,8 +263,7 @@ exclude = [
     "src/awkward/_typeparser/generated_parser.py",
 ]
 ignore = [
-    "E501",
-    "C408",
+    "E501",   # line too long
     "UP030",  # {0} -> {}
 ]
 select = [

diff --git a/src/awkward/highlevel.py b/src/awkward/highlevel.py
@@ -1016,7 +1016,7 @@ def __setitem__(self, where, what):
         """
         with ak._errors.OperationErrorContext(
             "ak.Array.__setitem__",
-            dict(self=self, field_name=where, field_value=what),
+            {"self": self, "field_name": where, "field_value": what},
         ):
             if not (
                 isinstance(where, str)
@@ -1049,7 +1049,7 @@ def __delitem__(self, where):
         """
         with ak._errors.OperationErrorContext(
             "ak.Array.__delitem__",
-            dict(self=self, field_name=where),
+            {"self": self, "field_name": where},
         ):
             if not (
                 isinstance(where, str)
@@ -1765,7 +1765,7 @@ def __setitem__(self, where, what):
         """
         with ak._errors.OperationErrorContext(
             "ak.Record.__setitem__",
-            dict(self=self, field_name=where, field_value=what),
+            {"self": self, "field_name": where, "field_value": what},
         ):
             if not (
                 isinstance(where, str)
@@ -1799,7 +1799,7 @@ def __delitem__(self, where):
         """
         with ak._errors.OperationErrorContext(
             "ak.Record.__delitem__",
-            dict(self=self, field_name=where),
+            {"self": self, "field_name": where},
         ):
             if not (
                 isinstance(where, str)

diff --git a/src/awkward/operations/ak_all.py b/src/awkward/operations/ak_all.py
@@ -52,14 +52,14 @@ def all(
     """
     with ak._errors.OperationErrorContext(
         "ak.all",
-        dict(
-            array=array,
-            axis=axis,
-            keepdims=keepdims,
-            mask_identity=mask_identity,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "array": array,
+            "axis": axis,
+            "keepdims": keepdims,
+            "mask_identity": mask_identity,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         if flatten_records is not unset:
             message = (

diff --git a/src/awkward/operations/ak_any.py b/src/awkward/operations/ak_any.py
@@ -52,14 +52,14 @@ def any(
     """
     with ak._errors.OperationErrorContext(
         "ak.any",
-        dict(
-            array=array,
-            axis=axis,
-            keepdims=keepdims,
-            mask_identity=mask_identity,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "array": array,
+            "axis": axis,
+            "keepdims": keepdims,
+            "mask_identity": mask_identity,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         if flatten_records is not unset:
             message = (

diff --git a/src/awkward/operations/ak_argcartesian.py b/src/awkward/operations/ak_argcartesian.py
@@ -84,15 +84,15 @@ def argcartesian(
     """
     with ak._errors.OperationErrorContext(
         "ak.argcartesian",
-        dict(
-            arrays=arrays,
-            axis=axis,
-            nested=nested,
-            parameters=parameters,
-            with_name=with_name,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "arrays": arrays,
+            "axis": axis,
+            "nested": nested,
+            "parameters": parameters,
+            "with_name": with_name,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         return _impl(arrays, axis, nested, parameters, with_name, highlevel, behavior)
 

diff --git a/src/awkward/operations/ak_argcombinations.py b/src/awkward/operations/ak_argcombinations.py
@@ -54,17 +54,17 @@ def argcombinations(
     """
     with ak._errors.OperationErrorContext(
         "ak.argcombinations",
-        dict(
-            array=array,
-            n=n,
-            replacement=replacement,
-            axis=axis,
-            fields=fields,
-            parameters=parameters,
-            with_name=with_name,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "array": array,
+            "n": n,
+            "replacement": replacement,
+            "axis": axis,
+            "fields": fields,
+            "parameters": parameters,
+            "with_name": with_name,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         return _impl(
             array,

diff --git a/src/awkward/operations/ak_argmax.py b/src/awkward/operations/ak_argmax.py
@@ -59,14 +59,14 @@ def argmax(
     """
     with ak._errors.OperationErrorContext(
         "ak.argmax",
-        dict(
-            array=array,
-            axis=axis,
-            keepdims=keepdims,
-            mask_identity=mask_identity,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "array": array,
+            "axis": axis,
+            "keepdims": keepdims,
+            "mask_identity": mask_identity,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         if flatten_records is not unset:
             message = (
@@ -123,14 +123,14 @@ def nanargmax(
     """
     with ak._errors.OperationErrorContext(
         "ak.nanargmax",
-        dict(
-            array=array,
-            axis=axis,
-            keepdims=keepdims,
-            mask_identity=mask_identity,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "array": array,
+            "axis": axis,
+            "keepdims": keepdims,
+            "mask_identity": mask_identity,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         if flatten_records is not unset:
             message = (

diff --git a/src/awkward/operations/ak_argmin.py b/src/awkward/operations/ak_argmin.py
@@ -59,14 +59,14 @@ def argmin(
     """
     with ak._errors.OperationErrorContext(
         "ak.argmin",
-        dict(
-            array=array,
-            axis=axis,
-            keepdims=keepdims,
-            mask_identity=mask_identity,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "array": array,
+            "axis": axis,
+            "keepdims": keepdims,
+            "mask_identity": mask_identity,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         if flatten_records is not unset:
             message = (
@@ -122,14 +122,14 @@ def nanargmin(
     """
     with ak._errors.OperationErrorContext(
         "ak.nanargmin",
-        dict(
-            array=array,
-            axis=axis,
-            keepdims=keepdims,
-            mask_identity=mask_identity,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "array": array,
+            "axis": axis,
+            "keepdims": keepdims,
+            "mask_identity": mask_identity,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         if flatten_records is not unset:
             message = (

diff --git a/src/awkward/operations/ak_argsort.py b/src/awkward/operations/ak_argsort.py
@@ -47,14 +47,14 @@ def argsort(
     """
     with ak._errors.OperationErrorContext(
         "ak.argsort",
-        dict(
-            array=array,
-            axis=axis,
-            ascending=ascending,
-            stable=stable,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "array": array,
+            "axis": axis,
+            "ascending": ascending,
+            "stable": stable,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         return _impl(array, axis, ascending, stable, highlevel, behavior)
 

diff --git a/src/awkward/operations/ak_broadcast_arrays.py b/src/awkward/operations/ak_broadcast_arrays.py
@@ -174,15 +174,15 @@ def broadcast_arrays(
     """
     with ak._errors.OperationErrorContext(
         "ak.broadcast_arrays",
-        dict(
-            arrays=arrays,
-            depth_limit=depth_limit,
-            broadcast_parameters_rule=broadcast_parameters_rule,
-            left_broadcast=left_broadcast,
-            right_broadcast=right_broadcast,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "arrays": arrays,
+            "depth_limit": depth_limit,
+            "broadcast_parameters_rule": broadcast_parameters_rule,
+            "left_broadcast": left_broadcast,
+            "right_broadcast": right_broadcast,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         return _impl(
             arrays,

diff --git a/src/awkward/operations/ak_cartesian.py b/src/awkward/operations/ak_cartesian.py
@@ -191,15 +191,15 @@ def cartesian(
     """
     with ak._errors.OperationErrorContext(
         "ak.cartesian",
-        dict(
-            arrays=arrays,
-            axis=axis,
-            nested=nested,
-            parameters=parameters,
-            with_name=with_name,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "arrays": arrays,
+            "axis": axis,
+            "nested": nested,
+            "parameters": parameters,
+            "with_name": with_name,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         return _impl(arrays, axis, nested, parameters, with_name, highlevel, behavior)
 

diff --git a/src/awkward/operations/ak_categories.py b/src/awkward/operations/ak_categories.py
@@ -18,7 +18,7 @@ def categories(array, highlevel=True):
     """
     with ak._errors.OperationErrorContext(
         "ak.categories",
-        dict(array=array, highlevel=highlevel),
+        {"array": array, "highlevel": highlevel},
     ):
         return _impl(array, highlevel)
 

diff --git a/src/awkward/operations/ak_combinations.py b/src/awkward/operations/ak_combinations.py
@@ -175,17 +175,17 @@ def combinations(
     """
     with ak._errors.OperationErrorContext(
         "ak.combinations",
-        dict(
-            array=array,
-            n=n,
-            replacement=replacement,
-            axis=axis,
-            fields=fields,
-            parameters=parameters,
-            with_name=with_name,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "array": array,
+            "n": n,
+            "replacement": replacement,
+            "axis": axis,
+            "fields": fields,
+            "parameters": parameters,
+            "with_name": with_name,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         return _impl(
             array,

diff --git a/src/awkward/operations/ak_concatenate.py b/src/awkward/operations/ak_concatenate.py
@@ -34,13 +34,13 @@ def concatenate(arrays, axis=0, *, mergebool=True, highlevel=True, behavior=None
     """
     with ak._errors.OperationErrorContext(
         "ak.concatenate",
-        dict(
-            arrays=arrays,
-            axis=axis,
-            mergebool=mergebool,
-            highlevel=highlevel,
-            behavior=behavior,
-        ),
+        {
+            "arrays": arrays,
+            "axis": axis,
+            "mergebool": mergebool,
+            "highlevel": highlevel,
+            "behavior": behavior,
+        },
     ):
         return _impl(arrays, axis, mergebool, highlevel, behavior)
 

diff --git a/src/awkward/operations/ak_copy.py b/src/awkward/operations/ak_copy.py
@@ -59,7 +59,7 @@ def copy(array):
     """
     with ak._errors.OperationErrorContext(
         "ak.fill_none",
-        dict(array=array),
+        {"array": array},
     ):
         return _impl(array)