We don't need function num_tiles_xy.
authorPhilipp Spitzer <philipp@spitzer.priv.at>
Fri, 28 Jun 2019 19:55:00 +0000 (21:55 +0200)
committerPhilipp Spitzer <philipp@spitzer.priv.at>
Fri, 28 Jun 2019 20:52:51 +0000 (22:52 +0200)
gimp_passport.py
test_gimp_passport.py

index 925e483..7fe8b3a 100644 (file)
@@ -48,19 +48,6 @@ def num_tiles(paper_size: int, tile_size: int, margin: int, spacing: int) -> int
     return max(0, int((size + spacing) // (tile_size + spacing)))
 
 
-def num_tiles_xy(paper_size: Tuple[int, int], tile_size: Tuple[int, int],
-                 margin: int, spacing: int) -> Tuple[int, int]:
-    """Returns the number of tiles that fit in the space described by the parameters.
-    All length units have to be the same (e.g. millimeter or pixel).
-
-    :param paper_size: tuple with paper width and height
-    :param tile_size: tuple with tile width and height
-    :param margin: space between paper border and first tile
-    :param spacing: space between tiles
-    """
-    return tuple(num_tiles(ps, ts, margin, spacing) for ps, ts in zip(paper_size, tile_size))
-
-
 def tile_edge(index: int, length: int, margin: int, spacing: int) -> int:
     """Returns the left (or top) pixel position (zero based) of the tile with index i.
     All length units have to be the same (e.g. millimeter or pixel).
index da2b366..a8eba4e 100644 (file)
@@ -1,7 +1,7 @@
 import os
 import unittest
 from PIL import Image, ImageDraw
-from gimp_passport import mm_to_pixel, num_tiles, num_tiles_xy, tile_edge, tile_bbox, line, make_passport, tile, \
+from gimp_passport import mm_to_pixel, num_tiles, tile_edge, tile_bbox, line, make_passport, tile, \
     downsample_large, main
 
 
@@ -31,12 +31,6 @@ class TestMakePassport(unittest.TestCase):
         self.assertEqual(3, num_tiles(30., 9., 1., 0.))
         self.assertEqual(2, num_tiles(30., 9., 1., 1.))
 
-    def test_num_tiles_xy(self):
-        self.assertEqual((3, 2), num_tiles_xy((45., 10.), (15., 5.), 0., 0.))
-        self.assertEqual((3, 2), num_tiles_xy((45., 10.), (15., 4.), 0., 0.))
-        self.assertEqual((2, 2), num_tiles_xy((45., 10.), (15., 4.), 0., 2.))
-        self.assertEqual((2, 1), num_tiles_xy((45., 10.), (15., 5.), 1., 1.))
-
     def test_tile_edge(self):
         margin = 4
         spacing = 3