ruff - enable and fix a series of UPNNN rules (#7748)

New ruff linter rules activated and fixed here:

- useless-object-inheritance (UP004)
- native-literals (UP018)
- yield-in-for-loop (UP028)
- extraneous-parentheses (UP034)
- deprecated-import (UP035)

Partially implements #7505

Author: pavoljuhas

benchmarks/circuit_construction.py

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 import itertools
-from typing import Sequence
+from collections.abc import Sequence
 
 import cirq
 

benchmarks/randomized_benchmarking.py

@@ -13,7 +13,7 @@
 # limitations under the License.
 
 import functools
-from typing import Sequence
+from collections.abc import Sequence
 
 import numpy as np
 

cirq-aqt/cirq_aqt/aqt_device.py

@@ -27,8 +27,9 @@
 from __future__ import annotations
 
 import json
+from collections.abc import Iterable, Sequence
 from enum import Enum
-from typing import Any, cast, Iterable, Sequence
+from typing import Any, cast
 
 import networkx as nx
 import numpy as np

cirq-aqt/cirq_aqt/aqt_device_metadata.py

@@ -16,7 +16,8 @@
 
 from __future__ import annotations
 
-from typing import Any, Iterable, Mapping
+from collections.abc import Iterable, Mapping
+from typing import Any
 
 import networkx as nx
 

cirq-aqt/cirq_aqt/aqt_sampler.py

@@ -27,7 +27,8 @@
 import json
 import time
 import uuid
-from typing import Callable, cast, Literal, Sequence, TypedDict
+from collections.abc import Callable, Sequence
+from typing import cast, Literal, TypedDict
 from urllib.parse import urljoin
 
 import numpy as np

cirq-core/cirq/_compat.py

@@ -27,8 +27,9 @@
 import sys
 import traceback
 import warnings
+from collections.abc import Callable, Iterator
 from types import ModuleType
-from typing import Any, Callable, Iterator, overload, TypeVar
+from typing import Any, overload, TypeVar
 
 import numpy as np
 import pandas as pd

cirq-core/cirq/_compat_test.py

@@ -25,9 +25,10 @@
 import traceback
 import types
 import warnings
+from collections.abc import Callable
 from importlib.machinery import ModuleSpec
 from types import ModuleType
-from typing import Any, Callable
+from typing import Any
 from unittest import mock
 
 import duet
@@ -159,7 +160,7 @@ def f(a, b):
 
 
 def test_deprecated_with_property():
-    class AClass(object):
+    class AClass:
         def __init__(self, a):
             self.a = a
 

cirq-core/cirq/_import.py

@@ -16,12 +16,13 @@
 
 import importlib
 import sys
+from collections.abc import Callable
 from contextlib import contextmanager
 from importlib import abc
 from importlib.abc import Loader
 from importlib.machinery import ModuleSpec
 from types import ModuleType
-from typing import Any, Callable, cast
+from typing import Any, cast
 
 
 class InstrumentedFinder(abc.MetaPathFinder):

cirq-core/cirq/circuits/_bucket_priority_queue.py

@@ -14,7 +14,8 @@
 
 from __future__ import annotations
 
-from typing import Any, Generic, Iterable, Iterator, TypeVar
+from collections.abc import Iterable, Iterator
+from typing import Any, Generic, TypeVar
 
 TItem = TypeVar('TItem')
 

cirq-core/cirq/circuits/circuit.py

@@ -27,24 +27,18 @@
 import itertools
 import math
 from collections import defaultdict
-from types import NotImplementedType
-from typing import (
-    AbstractSet,
-    Any,
+from collections.abc import (
     Callable,
-    cast,
     Hashable,
     Iterable,
     Iterator,
     Mapping,
     MutableSequence,
-    overload,
-    Self,
     Sequence,
-    TYPE_CHECKING,
-    TypeVar,
-    Union,
+    Set,
 )
+from types import NotImplementedType
+from typing import Any, cast, overload, Self, TYPE_CHECKING, TypeVar, Union
 
 import networkx
 import numpy as np
@@ -1341,7 +1335,7 @@ def _is_parameterized_(self) -> bool:
             protocols.is_parameterized(tag) for tag in self.tags
         )
 
-    def _parameter_names_(self) -> AbstractSet[str]:
+    def _parameter_names_(self) -> Set[str]:
         op_params = {name for op in self.all_operations() for name in protocols.parameter_names(op)}
         tag_params = {name for tag in self.tags for name in protocols.parameter_names(tag)}
         return op_params | tag_params
@@ -1845,7 +1839,7 @@ def __init__(
         self._frozen: cirq.FrozenCircuit | None = None
         self._is_measurement: bool | None = None
         self._is_parameterized: bool | None = None
-        self._parameter_names: AbstractSet[str] | None = None
+        self._parameter_names: Set[str] | None = None
         if not contents:
             return
         flattened_contents = tuple(ops.flatten_to_ops_or_moments(contents))
@@ -1954,7 +1948,7 @@ def _is_parameterized_(self) -> bool:
             self._is_parameterized = super()._is_parameterized_()
         return self._is_parameterized
 
-    def _parameter_names_(self) -> AbstractSet[str]:
+    def _parameter_names_(self) -> Set[str]:
         if self._parameter_names is None:
             self._parameter_names = super()._parameter_names_()
         return self._parameter_names