diff --git a/tilecloud_chain/internal_mapcache.py b/tilecloud_chain/internal_mapcache.py index 001a114a2..30a52ccb7 100644 --- a/tilecloud_chain/internal_mapcache.py +++ b/tilecloud_chain/internal_mapcache.py @@ -184,11 +184,12 @@ def compute_tile(self, tile: Tile) -> bool: if tile_.error: _LOG.error("Tile %s %s in error: %s", tile_.tilecoord, tile_.formated_metadata, tile_.error) success = False - if tile_.data is None: + elif tile_.data is None: _LOG.error("Tile %s %s in error: no data", tile_.tilecoord, tile_.formated_metadata) success = False - _LOG.debug("Tile %s %s generated", tile_.tilecoord, tile_.formated_metadata) - self._cache_store.put_one(tile_) + else: + _LOG.debug("Tile %s %s generated", tile_.tilecoord, tile_.formated_metadata) + self._cache_store.put_one(tile_) return success @contextlib.contextmanager diff --git a/tilecloud_chain/multitilestore.py b/tilecloud_chain/multitilestore.py index 95988806e..1f6cc1fb1 100644 --- a/tilecloud_chain/multitilestore.py +++ b/tilecloud_chain/multitilestore.py @@ -116,12 +116,13 @@ def apply(key: tuple[str, str], tiles: Iterator[Tile]) -> Iterator[Tile]: def __str__(self) -> str: """Return a string representation of the object.""" - stores = {store.__class__.__name__ for store in self.stores.values() if store is not None} + stores = {str(store) for store in self.stores.values() if store is not None} return f"{self.__class__.__name__}({', '.join(stores)})" def __repr__(self) -> str: """Return a string representation of the object.""" - return str(self) + stores = {repr(store) for store in self.stores.values() if store is not None} + return f"{self.__class__.__name__}({', '.join(stores)})" @staticmethod def _get_layer(tile: Tile | None) -> tuple[str, str]: diff --git a/tilecloud_chain/timedtilestore.py b/tilecloud_chain/timedtilestore.py index e2acb90d5..699c93578 100644 --- a/tilecloud_chain/timedtilestore.py +++ b/tilecloud_chain/timedtilestore.py @@ -121,4 +121,8 @@ def get_cheap_bounding_pyramid(self) -> BoundingPyramid | None: def __str__(self) -> str: """Get string representation.""" - return f"tilecloud_chain.timedtilestore.TimedTileStoreWrapper: {self._tile_store}" + return f"{self.__class__.__name__}({self._tile_store}" + + def __repr__(self) -> str: + """Get string representation.""" + return f"{self.__class__.__name__}({self._tile_store!r})"