Enable Pyrefly type checking (#3474)

Ref #3473

Description:

Enables [Pyrefly](https://github.com/facebook/pyrefly), a fast type
checker. I ported the `mypy.ini` file to a `pyrefly.toml` using the
`pyrefly init` command and added `# pyrefly: ignore` comments to
suppress new type errors.

Since this PR already touches a lot of files due to the suppressions, I
opted to keep it focused on adding Pyrefly. If it's accepted, I'll
follow up with another PR to remove Mypy, update documentation and other
references, and clean up easy-to-fix suppressions.

Author: rchen152

.github/workflows/typing-checks.yml

@@ -64,6 +64,10 @@ jobs:
           uv pip install -r requirements-dev.txt
           uv pip install .
           uv pip install mypy
+          uv pip install pyrefly
 
       - name: Run MyPy type checking
         run: mypy
+
+      - name: Run Pyrefly type checking
+        run: pyrefly check

ignite/contrib/engines/common.py

@@ -265,6 +265,7 @@ def _setup_common_distrib_training_handlers(
 
         @trainer.on(Events.EPOCH_STARTED)
         def distrib_set_epoch(engine: Engine) -> None:
+            # pyrefly: ignore [missing-attribute]
             train_sampler.set_epoch(engine.state.epoch - 1)
 
 

ignite/contrib/engines/tbptt.py

@@ -117,5 +117,6 @@ def _update(engine: Engine, batch: Sequence[torch.Tensor]) -> float:
         return sum(loss_list) / len(loss_list)
 
     engine = Engine(_update)
+    # pyrefly: ignore [bad-argument-type]
     engine.register_events(*Tbptt_Events)
     return engine

ignite/contrib/handlers/base_logger.py

@@ -9,7 +9,7 @@
 removed_in = "0.6.0"
 deprecation_warning = (
     f"{__file__} has been moved to /ignite/handlers/base_logger.py"
-    + (f" and will be removed in version {removed_in}" if removed_in else "")
+    + f" and will be removed in version {removed_in}"
     + ".\n Please refer to the documentation for more details."
 )
 warnings.warn(deprecation_warning, DeprecationWarning, stacklevel=2)

ignite/contrib/handlers/clearml_logger.py

@@ -9,7 +9,7 @@
 removed_in = "0.6.0"
 deprecation_warning = (
     f"{__file__} has been moved to /ignite/handlers/clearml_logger.py"
-    + (f" and will be removed in version {removed_in}" if removed_in else "")
+    + f" and will be removed in version {removed_in}"
     + ".\n Please refer to the documentation for more details."
 )
 warnings.warn(deprecation_warning, DeprecationWarning, stacklevel=2)

ignite/contrib/handlers/lr_finder.py

@@ -9,7 +9,7 @@
 removed_in = "0.6.0"
 deprecation_warning = (
     f"{__file__} has been moved to /ignite/handlers/lr_finder.py"
-    + (f" and will be removed in version {removed_in}" if removed_in else "")
+    + f" and will be removed in version {removed_in}"
     + ".\n Please refer to the documentation for more details."
 )
 warnings.warn(deprecation_warning, DeprecationWarning, stacklevel=2)

ignite/contrib/handlers/mlflow_logger.py

@@ -9,7 +9,7 @@
 removed_in = "0.6.0"
 deprecation_warning = (
     f"{__file__} has been moved to /ignite/handlers/mlflow_logger.py"
-    + (f" and will be removed in version {removed_in}" if removed_in else "")
+    + f" and will be removed in version {removed_in}"
     + ".\n Please refer to the documentation for more details."
 )
 warnings.warn(deprecation_warning, DeprecationWarning, stacklevel=2)

ignite/contrib/handlers/neptune_logger.py

@@ -9,7 +9,7 @@
 removed_in = "0.6.0"
 deprecation_warning = (
     f"{__file__} has been moved to /ignite/handlers/neptune_logger.py"
-    + (f" and will be removed in version {removed_in}" if removed_in else "")
+    + f" and will be removed in version {removed_in}"
     + ".\n Please refer to the documentation for more details."
 )
 warnings.warn(deprecation_warning, DeprecationWarning, stacklevel=2)

ignite/contrib/handlers/param_scheduler.py

@@ -9,7 +9,7 @@
 removed_in = "0.6.0"
 deprecation_warning = (
     f"{__file__} has been moved to /ignite/handlers/param_scheduler.py"
-    + (f" and will be removed in version {removed_in}" if removed_in else "")
+    + f" and will be removed in version {removed_in}"
     + ".\n Please refer to the documentation for more details."
 )
 warnings.warn(deprecation_warning, DeprecationWarning, stacklevel=2)

ignite/contrib/handlers/polyaxon_logger.py

@@ -9,7 +9,7 @@
 removed_in = "0.6.0"
 deprecation_warning = (
     f"{__file__} has been moved to /ignite/handlers/polyaxon_logger.py"
-    + (f" and will be removed in version {removed_in}" if removed_in else "")
+    + f" and will be removed in version {removed_in}"
     + ".\n Please refer to the documentation for more details."
 )
 warnings.warn(deprecation_warning, DeprecationWarning, stacklevel=2)