Quotation cleanups

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

Author: bgilbert

.github/maintainer/mkmaintainer.py

@@ -23,5 +23,5 @@
                 'labels': ','.join(info.get('labels', [])),
             }
         )
-        url = f"https://github.com/{info['repo']}/issues/new?{args}"
-        fh.write(f"- [{info['link-text']}]({url})\n")
+        url = f'https://github.com/{info["repo"]}/issues/new?{args}'
+        fh.write(f'- [{info["link-text"]}]({url})\n')

examples/deepzoom/deepzoom_tile.py

@@ -231,7 +231,7 @@ def _tile_done(self) -> None:
         count, total = self._processed, self._dz.tile_count
         if count % 100 == 0 or count == total:
             print(
-                "Tiling %s: wrote %d/%d tiles"
+                'Tiling %s: wrote %d/%d tiles'
                 % (self._associated or 'slide', count, total),
                 end='\r',
                 file=sys.stderr,

openslide/__init__.py

@@ -457,9 +457,9 @@ def read_region(
         if self._image is None:
             raise ValueError('Cannot read from a closed slide')
         if level != 0:
-            raise OpenSlideError("Invalid level")
+            raise OpenSlideError('Invalid level')
         if ['fail' for s in size if s < 0]:
-            raise OpenSlideError(f"Size {size} must be non-negative")
+            raise OpenSlideError(f'Size {size} must be non-negative')
         # Any corner of the requested region may be outside the bounds of
         # the image.  Create a transparent tile of the correct size and
         # paste the valid part of the region into the correct location.
@@ -470,7 +470,7 @@ def read_region(
             max(0, min(l + s - 1, limit - 1))
             for l, s, limit in zip(location, size, self._image.size)
         ]
-        tile = Image.new("RGBA", size, (0,) * 4)
+        tile = Image.new('RGBA', size, (0,) * 4)
         if not [
             'fail' for tl, br in zip(image_topleft, image_bottomright) if br - tl < 0
         ]:  # "< 0" not a typo
@@ -500,12 +500,12 @@ def open_slide(filename: lowlevel.Filename) -> OpenSlide | ImageSlide:
 if __name__ == '__main__':
     import sys
 
-    print("OpenSlide vendor:", OpenSlide.detect_format(sys.argv[1]))
-    print("PIL format:", ImageSlide.detect_format(sys.argv[1]))
+    print('OpenSlide vendor:', OpenSlide.detect_format(sys.argv[1]))
+    print('PIL format:', ImageSlide.detect_format(sys.argv[1]))
     with open_slide(sys.argv[1]) as _slide:
-        print("Dimensions:", _slide.dimensions)
-        print("Levels:", _slide.level_count)
-        print("Level dimensions:", _slide.level_dimensions)
-        print("Level downsamples:", _slide.level_downsamples)
-        print("Properties:", _slide.properties)
-        print("Associated images:", _slide.associated_images)
+        print('Dimensions:', _slide.dimensions)
+        print('Levels:', _slide.level_count)
+        print('Level dimensions:', _slide.level_dimensions)
+        print('Level downsamples:', _slide.level_downsamples)
+        print('Properties:', _slide.properties)
+        print('Associated images:', _slide.associated_images)

openslide/deepzoom.py

@@ -208,10 +208,10 @@ def _get_tile_info(
     ) -> tuple[tuple[tuple[int, int], int, tuple[int, int]], tuple[int, int]]:
         # Check parameters
         if dz_level < 0 or dz_level >= self._dz_levels:
-            raise ValueError("Invalid level")
+            raise ValueError('Invalid level')
         for t, t_lim in zip(t_location, self._t_dimensions[dz_level]):
             if t < 0 or t >= t_lim:
-                raise ValueError("Invalid address")
+                raise ValueError('Invalid address')
 
         # Get preferred slide level
         slide_level = self._slide_from_dz_level[dz_level]

openslide/lowlevel.py

@@ -89,10 +89,10 @@ def try_load(names: list[str]) -> CDLL:
         except FileNotFoundError as exc:
             raise ModuleNotFoundError(
                 "Couldn't locate OpenSlide DLL. "
-                "Try `pip install openslide-bin`, "
+                'Try `pip install openslide-bin`, '
                 "or if you're using an OpenSlide binary package, "
                 "ensure you've called os.add_dll_directory(). "
-                "https://openslide.org/api/python/#installing"
+                'https://openslide.org/api/python/#installing'
             ) from exc
     elif platform.system() == 'Darwin':
         try:
@@ -107,8 +107,8 @@ def try_load(names: list[str]) -> CDLL:
             if lib is None:
                 raise ModuleNotFoundError(
                     "Couldn't locate OpenSlide dylib. "
-                    "Try `pip install openslide-bin`. "
-                    "https://openslide.org/api/python/#installing"
+                    'Try `pip install openslide-bin`. '
+                    'https://openslide.org/api/python/#installing'
                 ) from exc
             return cdll.LoadLibrary(lib)
     else:
@@ -117,8 +117,8 @@ def try_load(names: list[str]) -> CDLL:
         except OSError as exc:
             raise ModuleNotFoundError(
                 "Couldn't locate OpenSlide shared library. "
-                "Try `pip install openslide-bin`. "
-                "https://openslide.org/api/python/#installing"
+                'Try `pip install openslide-bin`. '
+                'https://openslide.org/api/python/#installing'
             ) from exc
 
 
@@ -170,11 +170,11 @@ def invalidate(self) -> None:
     @classmethod
     def from_param(cls, obj: _OpenSlide) -> _OpenSlide:
         if obj.__class__ != cls:
-            raise ValueError("Not an OpenSlide reference")
+            raise ValueError('Not an OpenSlide reference')
         if not obj._as_parameter_:
-            raise ValueError("Passing undefined slide object")
+            raise ValueError('Passing undefined slide object')
         if not obj._valid:
-            raise ValueError("Passing closed slide object")
+            raise ValueError('Passing closed slide object')
         return obj
 
 
@@ -193,9 +193,9 @@ def __del__(self) -> None:
     @classmethod
     def from_param(cls, obj: _OpenSlideCache) -> _OpenSlideCache:
         if obj.__class__ != cls:
-            raise ValueError("Not an OpenSlide cache reference")
+            raise ValueError('Not an OpenSlide cache reference')
         if not obj._as_parameter_:
-            raise ValueError("Passing undefined cache object")
+            raise ValueError('Passing undefined cache object')
         return obj
 
 
@@ -247,15 +247,15 @@ def from_param(cls, obj: int) -> c_size_t:
 
 
 def _load_image(buf: _Buffer, size: tuple[int, int]) -> Image.Image:
-    '''buf must be a mutable buffer.'''
+    """buf must be a mutable buffer."""
     _convert.argb2rgba(buf)
     return Image.frombuffer('RGBA', size, buf, 'raw', 'RGBA', 0, 1)
 
 
 # check for errors opening an image file and wrap the resulting handle
 def _check_open(result: int | None, _func: Any, _args: Any) -> _OpenSlide:
     if result is None:
-        raise OpenSlideUnsupportedFormatError("Unsupported or missing image file")
+        raise OpenSlideUnsupportedFormatError('Unsupported or missing image file')
     slide = _OpenSlide(c_void_p(result))
     err = get_error(slide)
     if err is not None:
@@ -303,7 +303,7 @@ def _check_name_list(result: _Pointer[c_char_p], func: Any, args: Any) -> list[s
 
 
 class _FunctionUnavailable:
-    '''Standin for a missing optional function.  Fails when called.'''
+    """Standin for a missing optional function.  Fails when called."""
 
     def __init__(self, minimum_version: str):
         self._minimum_version = minimum_version
@@ -434,7 +434,7 @@ def read_region(
         # OpenSlide would catch this, but not before we tried to allocate
         # a negative-size buffer
         raise OpenSlideError(
-            "negative width (%d) or negative height (%d) not allowed" % (w, h)
+            'negative width (%d) or negative height (%d) not allowed' % (w, h)
         )
     if w == 0 or h == 0:
         # Image.frombuffer() would raise an exception

tests/test_base.py

@@ -34,7 +34,7 @@ def test_open_slide(self) -> None:
             self.assertTrue(isinstance(osr, ImageSlide))
 
     def test_lowlevel_available(self) -> None:
-        '''Ensure all exported functions have an 'available' attribute.'''
+        """Ensure all exported functions have an 'available' attribute."""
         for name in dir(lowlevel):
             attr = getattr(lowlevel, name)
             # ignore classes and unexported functions

tests/test_deepzoom.py

@@ -87,7 +87,7 @@ def test_get_tile(self) -> None:
 
         def test_tile_color_profile(self) -> None:
             if self.CLASS is OpenSlide and not lowlevel.read_icc_profile.available:
-                self.skipTest("requires OpenSlide 4.0.0")
+                self.skipTest('requires OpenSlide 4.0.0')
             self.assertEqual(len(self.dz.get_tile(9, (1, 0)).info['icc_profile']), 588)
 
         def test_get_tile_bad_level(self) -> None:

tests/test_imageslide.py

@@ -141,7 +141,7 @@ def test_read_region_bad_size(self) -> None:
     def test_thumbnail(self) -> None:
         self.assertEqual(self.osr.get_thumbnail((100, 100)).size, (100, 83))
 
-    @unittest.skipUnless(lowlevel.cache_create.available, "requires OpenSlide 4.0.0")
+    @unittest.skipUnless(lowlevel.cache_create.available, 'requires OpenSlide 4.0.0')
     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))

tests/test_openslide.py

@@ -36,7 +36,7 @@
 
 
 class TestCache(unittest.TestCase):
-    @unittest.skipUnless(lowlevel.cache_create.available, "requires OpenSlide 4.0.0")
+    @unittest.skipUnless(lowlevel.cache_create.available, 'requires OpenSlide 4.0.0')
     def test_create_cache(self) -> None:
         OpenSlideCache(0)
         OpenSlideCache(1)
@@ -153,7 +153,7 @@ def test_properties(self) -> None:
         )
 
     @unittest.skipUnless(
-        lowlevel.read_icc_profile.available, "requires OpenSlide 4.0.0"
+        lowlevel.read_icc_profile.available, 'requires OpenSlide 4.0.0'
     )
     def test_color_profile(self) -> None:
         assert self.osr.color_profile is not None  # for type inference
@@ -192,7 +192,7 @@ def _test_read_region_2GB(self) -> None:
     def test_thumbnail(self) -> None:
         self.assertEqual(self.osr.get_thumbnail((100, 100)).size, (100, 83))
 
-    @unittest.skipUnless(lowlevel.cache_create.available, "requires OpenSlide 4.0.0")
+    @unittest.skipUnless(lowlevel.cache_create.available, 'requires OpenSlide 4.0.0')
     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))
@@ -236,7 +236,7 @@ def test_color_profile(self) -> None:
 # Requires DICOM support in OpenSlide.  Use associated image ICC support as
 # a proxy.
 @unittest.skipUnless(
-    lowlevel.read_associated_image_icc_profile.available, "requires OpenSlide 4.0.0"
+    lowlevel.read_associated_image_icc_profile.available, 'requires OpenSlide 4.0.0'
 )
 class TestDicomSlide(_Abstract.SlideTest):
     FILENAME = 'boxes_0.dcm'