Remove bogus statements from type stubs (#352)

Avasam · web-flow · commit a721313e9af5 · 2025-03-14T15:49:21.000-07:00
diff --git a/stubs/matplotlib/transforms.pyi b/stubs/matplotlib/transforms.pyi
@@ -26,7 +26,6 @@ class TransformNode:
 class BboxBase(TransformNode):
     is_bbox = ...
     is_affine = ...
-    if DEBUG: ...
 
     def frozen(self): ...
     def __array__(self, *args, **kwargs): ...
diff --git a/stubs/networkx/algorithms/isomorphism/matchhelpers.pyi b/stubs/networkx/algorithms/isomorphism/matchhelpers.pyi
@@ -31,11 +31,7 @@ categorical_edge_match = ...
 def categorical_multiedge_match(attr: str | ArrayLike, default): ...
 
 # Docstrings for categorical functions.
-categorical_node_match.__doc__ = ...
-categorical_edge_match.__doc__ = ...
 tmpdoc = ...
-tmpdoc = ...
-categorical_multiedge_match.__doc__ = ...
 
 numerical_doc: str = ...
 
@@ -56,11 +52,7 @@ def numerical_multiedge_match(
 ): ...
 
 # Docstrings for numerical functions.
-numerical_node_match.__doc__ = ...
-numerical_edge_match.__doc__ = ...
-tmpdoc = ...
 tmpdoc = ...
-numerical_multiedge_match.__doc__ = ...
 
 generic_doc: str = ...
 
@@ -69,7 +61,3 @@ def generic_node_match(attr: str | ArrayLike, default, op: ArrayLike | Callable)
 generic_edge_match = ...
 
 def generic_multiedge_match(attr: str | ArrayLike, default, op: ArrayLike | Callable): ...
-
-# Docstrings for numerical functions.
-generic_node_match.__doc__ = ...
-generic_edge_match.__doc__ = ...
diff --git a/stubs/networkx/algorithms/tree/branchings.pyi b/stubs/networkx/algorithms/tree/branchings.pyi
@@ -112,14 +112,6 @@ docstring_branching: str = ...
 
 docstring_arborescence = ...
 
-maximum_branching.__doc__ = ...
-
-minimum_branching.__doc__ = ...
-
-maximum_spanning_arborescence.__doc__ = ...
-
-minimum_spanning_arborescence.__doc__ = ...
-
 class ArborescenceIterator:
     def __init__(
         self,
diff --git a/stubs/networkx/classes/digraph.pyi b/stubs/networkx/classes/digraph.pyi
@@ -51,9 +51,6 @@ class DiGraph(Graph):
     # alias out_edges to edges
     @cached_property
     def out_edges(self) -> OutEdgeView: ...
-
-    out_edges.__doc__ = ...
-
     @cached_property
     def in_edges(self) -> InEdgeView: ...
     @cached_property
diff --git a/stubs/networkx/classes/multidigraph.pyi b/stubs/networkx/classes/multidigraph.pyi
@@ -33,9 +33,6 @@ class MultiDiGraph(MultiGraph, DiGraph):
     # alias out_edges to edges
     @cached_property
     def out_edges(self) -> OutMultiEdgeView: ...
-
-    out_edges.__doc__ = ...
-
     @cached_property
     def in_edges(self) -> InMultiEdgeView: ...
     @cached_property
diff --git a/stubs/networkx/readwrite/gexf.pyi b/stubs/networkx/readwrite/gexf.pyi
@@ -12,11 +12,7 @@ def generate_gexf(G: Graph, encoding="utf-8", prettyprint=True, version="1.2draf
 def read_gexf(path, node_type=None, relabel=False, version="1.2draft"): ...
 
 class GEXF:
-    versions: dict = ...
-    d: dict = ...
-    versions["1.1draft"] = ...
-    d: dict = ...
-    versions["1.2draft"] = ...
+    versions: dict
 
     def construct_types(self): ...
 
diff --git a/stubs/skimage/__init__.pyi b/stubs/skimage/__init__.pyi
@@ -6,7 +6,8 @@ from ._shared import lazy as lazy
 from ._shared.tester import PytestTester as PytestTester  # noqa
 from ._shared.version_requirements import ensure_python_version as ensure_python_version
 
-__getattr__, __lazy_dir__, _ = ...
+__getattr__ = ...
+__lazy_dir__ = ...
 
 def __dir__(): ...
 
diff --git a/stubs/skimage/data/__init__.pyi b/stubs/skimage/data/__init__.pyi
@@ -1,3 +1,5 @@
 from .._shared import lazy as lazy
 
-__getattr__, __dir__, __all__ = ...
+__getattr__ = ...
+__dir__ = ...
+__all__ = ...  # pyright: ignore[reportUnsupportedDunderAll] # TODO
diff --git a/stubs/skimage/data/_fetchers.pyi b/stubs/skimage/data/_fetchers.pyi
@@ -15,7 +15,8 @@ skimage_distribution_dir = ...
 def _has_hash(path, expected_hash): ...
 def create_image_fetcher(): ...
 
-image_fetcher, data_dir = ...
+image_fetcher = ...
+data_dir = ...
 
 def _skip_pytest_case_requiring_pooch(data_filename): ...
 def _fetch(data_filename): ...
diff --git a/stubs/sklearn/cross_decomposition/_pls.pyi b/stubs/sklearn/cross_decomposition/_pls.pyi
@@ -70,8 +70,6 @@ class PLSRegression(_PLS):
     x_weights_: ndarray = ...
 
     _parameter_constraints: ClassVar[dict] = ...
-    for param in ("deflation_mode", "mode", "algorithm"):
-        pass
 
     # This implementation provides the same results that 3 PLS packages
     # provided in the R language (R-project):
@@ -104,8 +102,6 @@ class PLSCanonical(_PLS):
     x_weights_: ndarray = ...
 
     _parameter_constraints: ClassVar[dict] = ...
-    for param in ("deflation_mode", "mode"):
-        pass
 
     # This implementation provides the same results that the "plspm" package
     # provided in the R language (R-project), using the function plsca(X, Y).
@@ -140,8 +136,6 @@ class CCA(_PLS):
     x_weights_: ndarray = ...
 
     _parameter_constraints: ClassVar[dict] = ...
-    for param in ("deflation_mode", "mode", "algorithm"):
-        pass
 
     def __init__(
         self,
diff --git a/stubs/sklearn/ensemble/_forest.pyi b/stubs/sklearn/ensemble/_forest.pyi
@@ -273,8 +273,6 @@ class RandomTreesEmbedding(TransformerMixin, BaseForest):
     estimator_: ExtraTreeRegressor = ...
 
     _parameter_constraints: ClassVar[dict] = ...
-    for param in ("max_features", "ccp_alpha", "splitter"):
-        pass
 
     criterion: ClassVar[str] = ...
     max_features: ClassVar[int] = ...
diff --git a/stubs/sklearn/linear_model/_coordinate_descent.pyi b/stubs/sklearn/linear_model/_coordinate_descent.pyi
@@ -263,8 +263,6 @@ class MultiTaskElasticNet(Lasso):
     intercept_: ndarray = ...
 
     _parameter_constraints: ClassVar[dict] = ...
-    for param in ("precompute", "positive"):
-        pass
 
     def __init__(
         self,
diff --git a/stubs/sklearn/linear_model/_least_angle.pyi b/stubs/sklearn/linear_model/_least_angle.pyi
@@ -146,9 +146,6 @@ class LarsCV(Lars):
 
     _parameter_constraints: ClassVar[dict] = ...
 
-    for parameter in ["n_nonzero_coefs", "jitter", "fit_path", "random_state"]:
-        pass
-
     method: ClassVar[str] = ...
 
     def __init__(
@@ -213,9 +210,6 @@ class LassoLarsIC(LassoLars):
 
     _parameter_constraints: ClassVar[dict] = ...
 
-    for parameter in ["jitter", "fit_path", "alpha", "random_state"]:
-        pass
-
     def __init__(
         self,
         criterion: Literal["aic", "bic"] = "aic",
diff --git a/stubs/sklearn/linear_model/_logistic.pyi b/stubs/sklearn/linear_model/_logistic.pyi
@@ -99,9 +99,6 @@ class LogisticRegressionCV(LogisticRegression, LinearClassifierMixin, BaseEstima
 
     _parameter_constraints: ClassVar[dict] = ...
 
-    for param in ["C", "warm_start", "l1_ratio"]:
-        pass
-
     def __init__(
         self,
         *,
diff --git a/stubs/sklearn/linear_model/_ridge.pyi b/stubs/sklearn/linear_model/_ridge.pyi
@@ -227,8 +227,6 @@ class RidgeClassifierCV(_RidgeClassifierMixin, _BaseRidgeCV):
     cv_values_: ndarray = ...
 
     _parameter_constraints: ClassVar[dict] = ...
-    for param in ("gcv_mode", "alpha_per_target"):
-        pass
 
     def __init__(
         self,
diff --git a/stubs/sklearn/svm/_base.pyi b/stubs/sklearn/svm/_base.pyi
@@ -70,8 +70,6 @@ class BaseLibSVM(BaseEstimator, metaclass=ABCMeta):
 
 class BaseSVC(ClassifierMixin, BaseLibSVM, metaclass=ABCMeta):
     _parameter_constraints: ClassVar[dict] = ...
-    for unused_param in ["epsilon", "nu"]:
-        pass
 
     @abstractmethod
     def __init__(
diff --git a/stubs/sklearn/svm/_classes.pyi b/stubs/sklearn/svm/_classes.pyi
@@ -177,8 +177,6 @@ class SVR(RegressorMixin, BaseLibSVM):
     _impl: ClassVar[str] = ...
 
     _parameter_constraints: ClassVar[dict] = ...
-    for unused_param in ["class_weight", "nu", "probability", "random_state"]:
-        pass
 
     def __init__(
         self,
@@ -217,8 +215,6 @@ class NuSVR(RegressorMixin, BaseLibSVM):
     _impl: ClassVar[str] = ...
 
     _parameter_constraints: ClassVar[dict] = ...
-    for unused_param in ["class_weight", "epsilon", "probability", "random_state"]:
-        pass
 
     def __init__(
         self,
@@ -258,8 +254,6 @@ class OneClassSVM(OutlierMixin, BaseLibSVM):
     _impl: ClassVar[str] = ...
 
     _parameter_constraints: ClassVar[dict] = ...
-    for unused_param in ["C", "class_weight", "epsilon", "probability", "random_state"]:
-        pass
 
     def __init__(
         self,
diff --git a/stubs/sympy-stubs/__init__.pyi b/stubs/sympy-stubs/__init__.pyi
@@ -921,9 +921,7 @@ from sympy.utilities import (
     xthreaded,
 )
 
-if sys.version_info < (3, 8): ...
-if "dev" in __version__:
-    def enable_warnings() -> None: ...
+def enable_warnings() -> None: ...
 
 SYMPY_DEBUG: bool = ...
 test = ...
diff --git a/stubs/sympy-stubs/core/cache.pyi b/stubs/sympy-stubs/core/cache.pyi
@@ -9,14 +9,8 @@ print_cache = ...
 clear_cache = ...
 USE_CACHE = ...
 scs = ...
-if scs.lower() == "none":
-    SYMPY_CACHE_SIZE = ...
-else:
-    SYMPY_CACHE_SIZE = ...
-if USE_CACHE == "no":
-    cacheit = ...
-else:
-    cacheit = ...
+SYMPY_CACHE_SIZE = ...
+cacheit = ...
 
 def cached_property(func) -> property: ...
 def lazy_function(module: str, name: str) -> Callable:
diff --git a/stubs/sympy-stubs/core/relational.pyi b/stubs/sympy-stubs/core/relational.pyi
@@ -100,7 +100,6 @@ class GreaterThan(_Greater):
 Ge = GreaterThan
 
 class LessThan(_Less):
-    __doc__ = ...
     __slots__ = ...
     rel_op = ...
     @property
@@ -109,7 +108,6 @@ class LessThan(_Less):
 Le = LessThan
 
 class StrictGreaterThan(_Greater):
-    __doc__ = ...
     __slots__ = ...
     rel_op = ...
     @property
@@ -118,7 +116,6 @@ class StrictGreaterThan(_Greater):
 Gt = StrictGreaterThan
 
 class StrictLessThan(_Less):
-    __doc__ = ...
     __slots__ = ...
     rel_op = ...
     @property
diff --git a/stubs/sympy-stubs/physics/control/control_plots.pyi b/stubs/sympy-stubs/physics/control/control_plots.pyi
@@ -17,10 +17,6 @@ __all__ = [
 ]
 matplotlib = ...
 numpy = ...
-if matplotlib:
-    plt = ...
-if numpy:
-    np = ...
 
 def pole_zero_numerical_data(system) -> tuple[Any, Any]: ...
 def pole_zero_plot(
diff --git a/stubs/sympy-stubs/plotting/pygletplot/managed_window.pyi b/stubs/sympy-stubs/plotting/pygletplot/managed_window.pyi
@@ -9,5 +9,3 @@ class ManagedWindow:
     def setup(self) -> None: ...
     def update(self, dt) -> None: ...
     def draw(self) -> None: ...
-
-if __name__ == "__main__": ...
diff --git a/stubs/sympy-stubs/polys/numberfields/galois_resolvents.pyi b/stubs/sympy-stubs/polys/numberfields/galois_resolvents.pyi
@@ -26,8 +26,3 @@ def define_resolvents() -> dict[
 ]: ...
 def generate_lambda_lookup(verbose=..., trial_run=...) -> str: ...
 def get_resolvent_by_lookup(T, number) -> list[Any]: ...
-
-if __name__ == "__main__":
-    verbose = ...
-    trial_run = ...
-    table = ...
diff --git a/stubs/sympy-stubs/simplify/fu.pyi b/stubs/sympy-stubs/simplify/fu.pyi
@@ -29,7 +29,6 @@ def TR22(rv, max=..., pow=...): ...
 def TRpower(rv): ...
 def L(rv): ...
 
-if SYMPY_DEBUG: ...
 CTR1 = ...
 CTR2 = ...
 CTR3 = ...
diff --git a/stubs/sympy-stubs/testing/runtests.pyi b/stubs/sympy-stubs/testing/runtests.pyi
@@ -13,8 +13,6 @@ class Skipped(Exception): ...
 class TimeOutError(Exception): ...
 class DependencyError(Exception): ...
 
-if IS_WINDOWS: ...
-
 def convert_to_native_paths(lst) -> list[Any]: ...
 def get_sympy_dir() -> str: ...
 def setup_pprint() -> bool: ...
diff --git a/stubs/vispy/ext/cocoapy.pyi b/stubs/vispy/ext/cocoapy.pyi
diff --git a/stubs/vispy/scene/cameras/magnify.pyi b/stubs/vispy/scene/cameras/magnify.pyi
diff --git a/stubs/vispy/util/logs.pyi b/stubs/vispy/util/logs.pyi
diff --git a/tests/run_hygiene.py b/tests/run_hygiene.py
