Skip to content

Commit c26773d

Browse files
authored
fix: Fix HeatmapLayer by bumping deck.gl to 9.2 (#910)
Working again! <img width="808" height="557" alt="image" src="https://github.com/user-attachments/assets/11f917ac-9c5b-4801-9d4a-77eef56601c3" /> Thanks @felixpalmer! Closes #646
1 parent e304440 commit c26773d

File tree

6 files changed

+241
-294
lines changed

6 files changed

+241
-294
lines changed

lonboard/_layer.py

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@
1313

1414
from __future__ import annotations
1515

16-
import warnings
17-
from textwrap import dedent
1816
from typing import TYPE_CHECKING, Any
1917

2018
import ipywidgets
@@ -2030,21 +2028,9 @@ def __init__(
20302028
table: ArrowStreamExportable,
20312029
**kwargs: Unpack[HeatmapLayerKwargs],
20322030
) -> None:
2033-
err_msg = """
2034-
The `HeatmapLayer` is not currently working.
2035-
2036-
As of Lonboard v0.10, Lonboard upgraded to version 9.0 of the underlying
2037-
[deck.gl](https://deck.gl/) library. deck.gl [appears to have a
2038-
bug](https://github.com/visgl/deck.gl/issues/8960#issuecomment-2284791644) with
2039-
the HeatmapLayer in 9.0, that has not yet been fixed.
2040-
2041-
Please temporarily downgrade to Lonboard v0.9 if you would like to use the
2042-
`HeatmapLayer`.
2043-
"""
2044-
warnings.warn(dedent(err_msg), UserWarning)
2045-
20462031
# NOTE: we override the default for _rows_per_chunk because otherwise we render
2047-
# one heatmap per _chunk_ not for the entire dataset.
2032+
# one heatmap per _chunk_ not for the entire dataset (we don't have a way to
2033+
# concat batches in the frontend)
20482034
table_o3 = Table.from_arrow(table)
20492035
super().__init__(table=table, _rows_per_chunk=len(table_o3), **kwargs)
20502036

0 commit comments

Comments
 (0)