update docs build reqs

[hackapet/Adafruit_Blinka_Displayio.git] / displayio / _tilegrid.py

diff --git a/displayio/_tilegrid.py b/displayio/_tilegrid.py
index 9287e980a64a71f908c0b3808018c7a579532413..a3f21eb53286aa835fdd9aa6b03b2a5000f8bacd 100644 (file)
--- a/displayio/_tilegrid.py
+++ b/displayio/_tilegrid.py
@@ -23,7 +23,6 @@ from circuitpython_typing import WriteableBuffer
 from ._bitmap import Bitmap
 from ._colorconverter import ColorConverter
 from ._ondiskbitmap import OnDiskBitmap
 from ._bitmap import Bitmap
 from ._colorconverter import ColorConverter
 from ._ondiskbitmap import OnDiskBitmap

-from ._shape import Shape

 from ._palette import Palette
 from ._structs import (
     InputPixelStruct,
 from ._palette import Palette
 from ._structs import (
     InputPixelStruct,


@@ -47,7 +46,7 @@ class TileGrid:
 
     def __init__(
         self,
 
     def __init__(
         self,

-        bitmap: Union[Bitmap, OnDiskBitmap, Shape],
+        bitmap: Union[Bitmap, OnDiskBitmap],

         *,
         pixel_shader: Union[ColorConverter, Palette],
         width: int = 1,
         *,
         pixel_shader: Union[ColorConverter, Palette],
         width: int = 1,


@@ -64,7 +63,7 @@ class TileGrid:
 
         tile_width and tile_height match the height of the bitmap by default.
         """
 
         tile_width and tile_height match the height of the bitmap by default.
         """

-        if not isinstance(bitmap, (Bitmap, OnDiskBitmap, Shape)):
+        if not isinstance(bitmap, (Bitmap, OnDiskBitmap)):

             raise ValueError("Unsupported Bitmap type")
         self._bitmap = bitmap
         bitmap_width = bitmap.width
             raise ValueError("Unsupported Bitmap type")
         self._bitmap = bitmap
         bitmap_width = bitmap.width


@@ -121,6 +120,7 @@ class TileGrid:
         self._tiles_in_bitmap = self._bitmap_width_in_tiles * (
             bitmap_height // tile_height
         )
         self._tiles_in_bitmap = self._bitmap_width_in_tiles * (
             bitmap_height // tile_height
         )

+        self._needs_refresh = True

 
     def _update_transform(self, absolute_transform):
         """Update the parent transform and child transforms"""
 
     def _update_transform(self, absolute_transform):
         """Update the parent transform and child transforms"""


@@ -132,6 +132,7 @@ class TileGrid:
             self._update_current_y()
 
     def _update_current_x(self):
             self._update_current_y()
 
     def _update_current_x(self):

+        self._needs_refresh = True

         if self._transpose_xy:
             width = self._pixel_height
         else:
         if self._transpose_xy:
             width = self._pixel_height
         else:


@@ -169,6 +170,7 @@ class TileGrid:
                 )
 
     def _update_current_y(self):
                 )
 
     def _update_current_y(self):

+        self._needs_refresh = True

         if self._transpose_xy:
             height = self._pixel_width
         else:
         if self._transpose_xy:
             height = self._pixel_width
         else:


@@ -341,7 +343,7 @@ class TileGrid:
 
                 # We always want to read bitmap pixels by row first and then transpose into
                 # the destination buffer because most bitmaps are row associated.
 
                 # We always want to read bitmap pixels by row first and then transpose into
                 # the destination buffer because most bitmaps are row associated.

-                if isinstance(self._bitmap, (Bitmap, Shape, OnDiskBitmap)):
+                if isinstance(self._bitmap, (Bitmap, OnDiskBitmap)):

                     input_pixel.pixel = (
                         self._bitmap._get_pixel(  # pylint: disable=protected-access
                             input_pixel.tile_x, input_pixel.tile_y
                     input_pixel.pixel = (
                         self._bitmap._get_pixel(  # pylint: disable=protected-access
                             input_pixel.tile_x, input_pixel.tile_y


@@ -364,19 +366,18 @@ class TileGrid:
                     full_coverage = False
                 else:
                     mask[offset // 32] |= 1 << (offset % 32)
                     full_coverage = False
                 else:
                     mask[offset // 32] |= 1 << (offset % 32)

-                    # print("Mask", mask)

                     if colorspace.depth == 16:
                         struct.pack_into(
                             "H",
                     if colorspace.depth == 16:
                         struct.pack_into(
                             "H",

-                            buffer.cast("H"),
-                            offset,
+                            buffer.cast("B"),
+                            offset * 2,

                             output_pixel.pixel,
                         )
                     elif colorspace.depth == 32:
                         struct.pack_into(
                             "I",
                             output_pixel.pixel,
                         )
                     elif colorspace.depth == 32:
                         struct.pack_into(
                             "I",

-                            buffer,
-                            offset,
+                            buffer.cast("B"),
+                            offset * 4,

                             output_pixel.pixel,
                         )
                     elif colorspace.depth == 8:
                             output_pixel.pixel,
                         )
                     elif colorspace.depth == 8:


@@ -406,20 +407,21 @@ class TileGrid:
         return full_coverage
 
     def _finish_refresh(self):
         return full_coverage
 
     def _finish_refresh(self):

-        first_draw = self._previous_area.x1 == self._previous_area.x2
-        hidden = self._hidden_tilegrid or self._hidden_by_parent
-        if not first_draw and hidden:
-            self._previous_area.x2 = self._previous_area.x1
-        elif self._moved or first_draw:
-            self._current_area.copy_into(self._previous_area)
-
-        self._moved = False
-        self._full_change = False
-        self._partial_change = False
-        if isinstance(self._pixel_shader, (Palette, ColorConverter)):
-            self._pixel_shader._finish_refresh()  # pylint: disable=protected-access
-        if isinstance(self._bitmap, (Bitmap, Shape)):
-            self._bitmap._finish_refresh()  # pylint: disable=protected-access
+        if not self._needs_refresh:
+            first_draw = self._previous_area.x1 == self._previous_area.x2
+            hidden = self._hidden_tilegrid or self._hidden_by_parent
+            if not first_draw and hidden:
+                self._previous_area.x2 = self._previous_area.x1
+            elif self._moved or first_draw:
+                self._current_area.copy_into(self._previous_area)
+
+            self._moved = False
+            self._full_change = False
+            self._partial_change = False
+            if isinstance(self._pixel_shader, (Palette, ColorConverter)):
+                self._pixel_shader._finish_refresh()  # pylint: disable=protected-access
+            if isinstance(self._bitmap, Bitmap):
+                self._bitmap._finish_refresh()  # pylint: disable=protected-access

 
     def _get_refresh_areas(self, areas: list[Area]) -> None:
         # pylint: disable=invalid-name, too-many-branches, too-many-statements
 
     def _get_refresh_areas(self, areas: list[Area]) -> None:
         # pylint: disable=invalid-name, too-many-branches, too-many-statements


@@ -431,14 +433,17 @@ class TileGrid:
             self._rendered_hidden = True
             if not first_draw:
                 areas.append(self._previous_area)
             self._rendered_hidden = True
             if not first_draw:
                 areas.append(self._previous_area)

+            self._needs_refresh = False

             return
         if self._moved and not first_draw:
             self._previous_area.union(self._current_area, self._dirty_area)
             if self._dirty_area.size() < 2 * self._pixel_width * self._pixel_height:
                 areas.append(self._dirty_area)
             return
         if self._moved and not first_draw:
             self._previous_area.union(self._current_area, self._dirty_area)
             if self._dirty_area.size() < 2 * self._pixel_width * self._pixel_height:
                 areas.append(self._dirty_area)

+                self._needs_refresh = False

                 return
             areas.append(self._current_area)
             areas.append(self._previous_area)
                 return
             areas.append(self._current_area)
             areas.append(self._previous_area)

+            self._needs_refresh = False

             return
 
         tail = areas[-1] if areas else None
             return
 
         tail = areas[-1] if areas else None


@@ -454,12 +459,6 @@ class TileGrid:
                     self._partial_change = True
                 else:
                     self._full_change = True
                     self._partial_change = True
                 else:
                     self._full_change = True

-        elif isinstance(self._bitmap, Shape):
-            self._bitmap._get_refresh_areas(areas)  # pylint: disable=protected-access
-            refresh_area = areas[-1] if areas else None
-            if refresh_area != tail:
-                refresh_area.copy_into(self._dirty_area)
-                self._partial_change = True

 
         self._full_change = self._full_change or (
             isinstance(self._pixel_shader, (Palette, ColorConverter))
 
         self._full_change = self._full_change or (
             isinstance(self._pixel_shader, (Palette, ColorConverter))


@@ -467,6 +466,7 @@ class TileGrid:
         )
         if self._full_change or first_draw:
             areas.append(self._current_area)
         )
         if self._full_change or first_draw:
             areas.append(self._current_area)

+            self._needs_refresh = False

             return
 
         if self._partial_change:
             return
 
         if self._partial_change:


@@ -510,14 +510,17 @@ class TileGrid:
                     self._dirty_area.x1,
                 )
             areas.append(self._dirty_area)
                     self._dirty_area.x1,
                 )
             areas.append(self._dirty_area)

+        self._needs_refresh = False

 
     def _set_hidden(self, hidden: bool) -> None:
 
     def _set_hidden(self, hidden: bool) -> None:

+        self._needs_refresh = True

         self._hidden_tilegrid = hidden
         self._rendered_hidden = False
         if not hidden:
             self._full_change = True
 
     def _set_hidden_by_parent(self, hidden: bool) -> None:
         self._hidden_tilegrid = hidden
         self._rendered_hidden = False
         if not hidden:
             self._full_change = True
 
     def _set_hidden_by_parent(self, hidden: bool) -> None:

+        self._needs_refresh = True

         self._hidden_by_parent = hidden
         self._rendered_hidden = False
         if not hidden:
         self._hidden_by_parent = hidden
         self._rendered_hidden = False
         if not hidden:


@@ -536,6 +539,7 @@ class TileGrid:
         self._full_change = True
 
     def _set_tile(self, x: int, y: int, tile_index: int) -> None:
         self._full_change = True
 
     def _set_tile(self, x: int, y: int, tile_index: int) -> None:

+        self._needs_refresh = True

         self._tiles[y * self._width_in_tiles + x] = tile_index
         temp_area = Area()
         if not self._partial_change:
         self._tiles[y * self._width_in_tiles + x] = tile_index
         temp_area = Area()
         if not self._partial_change:


@@ -616,6 +620,7 @@ class TileGrid:
         if not isinstance(value, bool):
             raise TypeError("Flip X should be a boolean type")
         if self._flip_x != value:
         if not isinstance(value, bool):
             raise TypeError("Flip X should be a boolean type")
         if self._flip_x != value:

+            self._needs_refresh = True

             self._flip_x = value
             self._full_change = True
 
             self._flip_x = value
             self._full_change = True
 


@@ -629,6 +634,7 @@ class TileGrid:
         if not isinstance(value, bool):
             raise TypeError("Flip Y should be a boolean type")
         if self._flip_y != value:
         if not isinstance(value, bool):
             raise TypeError("Flip Y should be a boolean type")
         if self._flip_y != value:

+            self._needs_refresh = True

             self._flip_y = value
             self._full_change = True
 
             self._flip_y = value
             self._full_change = True
 


@@ -644,6 +650,7 @@ class TileGrid:
         if not isinstance(value, bool):
             raise TypeError("Transpose XY should be a boolean type")
         if self._transpose_xy != value:
         if not isinstance(value, bool):
             raise TypeError("Transpose XY should be a boolean type")
         if self._transpose_xy != value:

+            self._needs_refresh = True

             self._transpose_xy = value
             if self._pixel_width == self._pixel_height:
                 self._full_change = True
             self._transpose_xy = value
             if self._pixel_width == self._pixel_height:
                 self._full_change = True


@@ -668,21 +675,20 @@ class TileGrid:
 
         self._pixel_shader = new_pixel_shader
         self._full_change = True
 
         self._pixel_shader = new_pixel_shader
         self._full_change = True

+        self._needs_refresh = True

 
     @property
 
     @property

-    def bitmap(self) -> Union[Bitmap, OnDiskBitmap, Shape]:
-        """The Bitmap, OnDiskBitmap, or Shape that is assigned to this TileGrid"""
+    def bitmap(self) -> Union[Bitmap, OnDiskBitmap]:
+        """The Bitmap or OnDiskBitmap that is assigned to this TileGrid"""

         return self._bitmap
 
     @bitmap.setter
         return self._bitmap
 
     @bitmap.setter

-    def bitmap(self, new_bitmap: Union[Bitmap, OnDiskBitmap, Shape]) -> None:
-        if (
-            not isinstance(new_bitmap, Bitmap)
-            and not isinstance(new_bitmap, OnDiskBitmap)
-            and not isinstance(new_bitmap, Shape)
+    def bitmap(self, new_bitmap: Union[Bitmap, OnDiskBitmap]) -> None:
+        if not isinstance(new_bitmap, Bitmap) and not isinstance(
+            new_bitmap, OnDiskBitmap

         ):
             raise TypeError(
         ):
             raise TypeError(

-                "Unsupported Type: new_bitmap must be Bitmap, OnDiskBitmap, or Shape"
+                "Unsupported Type: new_bitmap must be Bitmap or OnDiskBitmap"

             )
 
         if (
             )
 
         if (


@@ -691,6 +697,7 @@ class TileGrid:
         ):
             raise ValueError("New bitmap must be same size as old bitmap")
 
         ):
             raise ValueError("New bitmap must be same size as old bitmap")
 

+        self._needs_refresh = True

         self._bitmap = new_bitmap
         self._full_change = True
 
         self._bitmap = new_bitmap
         self._full_change = True