Autofix and reformat with Ruff

Signed-off-by: Benjamin Gilbert <bgilbert@cs.cmu.edu>

Author: bgilbert

doc/jekyll_fix.py

@@ -38,7 +38,7 @@
 }
 FILES = {
     # Added in Sphinx 5.0.0, scheduled to be removed in Sphinx 6
-    'static/_sphinx_javascript_frameworks_compat.js': 'static/sphinx_javascript_frameworks_compat.js',  # noqa: E501
+    'static/_sphinx_javascript_frameworks_compat.js': 'static/sphinx_javascript_frameworks_compat.js',
 }
 REWRITE_EXTENSIONS = {'.html', '.js'}
 

examples/deepzoom/deepzoom_multiserver.py

@@ -31,8 +31,8 @@
 from typing import TYPE_CHECKING, Any, Literal
 import zlib
 
-from PIL import Image, ImageCms
 from flask import Flask, Response, abort, make_response, render_template, url_for
+from PIL import Image, ImageCms
 
 if TYPE_CHECKING:
     # Python 3.10+
@@ -41,7 +41,7 @@
 if os.name == 'nt':
     _dll_path = os.getenv('OPENSLIDE_PATH')
     if _dll_path is not None:
-        with os.add_dll_directory(_dll_path):  # type: ignore[attr-defined,unused-ignore]  # noqa: E501
+        with os.add_dll_directory(_dll_path):  # type: ignore[attr-defined,unused-ignore]
             import openslide
     else:
         import openslide

examples/deepzoom/deepzoom_server.py

@@ -31,8 +31,8 @@
 from unicodedata import normalize
 import zlib
 
-from PIL import Image, ImageCms
 from flask import Flask, Response, abort, make_response, render_template, url_for
+from PIL import Image, ImageCms
 
 if TYPE_CHECKING:
     # Python 3.10+
@@ -41,7 +41,7 @@
 if os.name == 'nt':
     _dll_path = os.getenv('OPENSLIDE_PATH')
     if _dll_path is not None:
-        with os.add_dll_directory(_dll_path):  # type: ignore[attr-defined,unused-ignore]  # noqa: E501
+        with os.add_dll_directory(_dll_path):  # type: ignore[attr-defined,unused-ignore]
             import openslide
     else:
         import openslide

examples/deepzoom/deepzoom_tile.py

@@ -47,7 +47,7 @@
 if os.name == 'nt':
     _dll_path = os.getenv('OPENSLIDE_PATH')
     if _dll_path is not None:
-        with os.add_dll_directory(_dll_path):  # type: ignore[attr-defined,unused-ignore]  # noqa: E501
+        with os.add_dll_directory(_dll_path):  # type: ignore[attr-defined,unused-ignore]
             import openslide
     else:
         import openslide

openslide/__init__.py

@@ -35,16 +35,12 @@
 from openslide import lowlevel
 
 # Re-exports for the benefit of library users
-from openslide._version import (  # noqa: F401  module-imported-but-unused
-    __version__ as __version__,
-)
+from openslide._version import __version__ as __version__
+from openslide.lowlevel import OpenSlideError as OpenSlideError
 from openslide.lowlevel import (
     OpenSlideUnsupportedFormatError as OpenSlideUnsupportedFormatError,
 )
-from openslide.lowlevel import (  # noqa: F401  module-imported-but-unused
-    OpenSlideVersionError as OpenSlideVersionError,
-)
-from openslide.lowlevel import OpenSlideError as OpenSlideError
+from openslide.lowlevel import OpenSlideVersionError as OpenSlideVersionError
 
 __library_version__ = lowlevel.get_version()
 

openslide/deepzoom.py

@@ -147,13 +147,7 @@ def tiles(z_lim: int) -> int:
         )
 
     def __repr__(self) -> str:
-        return '{}({!r}, tile_size={!r}, overlap={!r}, limit_bounds={!r})'.format(
-            self.__class__.__name__,
-            self._osr,
-            self._z_t_downsample,
-            self._z_overlap,
-            self._limit_bounds,
-        )
+        return f'{self.__class__.__name__}({self._osr!r}, tile_size={self._z_t_downsample!r}, overlap={self._z_overlap!r}, limit_bounds={self._limit_bounds!r})'
 
     @property
     def level_count(self) -> int:

tests/common.py

@@ -29,7 +29,7 @@
     # environment.
     _dll_path = os.getenv('OPENSLIDE_PATH')
     if _dll_path is not None:
-        with os.add_dll_directory(_dll_path):  # type: ignore[attr-defined,unused-ignore]  # noqa: E501
+        with os.add_dll_directory(_dll_path):  # type: ignore[attr-defined,unused-ignore]
             import openslide  # noqa: F401  module-imported-but-unused
 
 

tests/test_imageslide.py

@@ -21,8 +21,8 @@
 import sys
 import unittest
 
-from PIL import Image
 from common import file_path
+from PIL import Image
 
 from openslide import ImageSlide, OpenSlideCache, OpenSlideError, lowlevel
 

tests/test_openslide.py

@@ -43,7 +43,8 @@ def test_create_cache(self) -> None:
         OpenSlideCache(4 << 20)
         self.assertRaises(ArgumentError, lambda: OpenSlideCache(-1))
         self.assertRaises(
-            ArgumentError, lambda: OpenSlideCache(1.3)  # type: ignore[arg-type]
+            ArgumentError,
+            lambda: OpenSlideCache(1.3),  # type: ignore[arg-type]
         )
 
 
@@ -197,10 +198,12 @@ def test_set_cache(self) -> None:
         self.osr.set_cache(OpenSlideCache(64 << 10))
         self.assertEqual(self.osr.read_region((0, 0), 0, (400, 400)).size, (400, 400))
         self.assertRaises(
-            TypeError, lambda: self.osr.set_cache(None)  # type: ignore[arg-type]
+            TypeError,
+            lambda: self.osr.set_cache(None),  # type: ignore[arg-type]
         )
         self.assertRaises(
-            TypeError, lambda: self.osr.set_cache(3)  # type: ignore[arg-type]
+            TypeError,
+            lambda: self.osr.set_cache(3),  # type: ignore[arg-type]
         )