Skip to content

Commit 08d6973

Browse files
vgeorgekylebarron
andauthored
Upgrade Deck.gl to version 9 (#562)
Contributes to #422. This is a work in progress. The `north-america-roads` example works and type checks pass, but I haven't been able to test this thoroughly yet. cc @kylebarron @batpad --------- Co-authored-by: Kyle Barron <kyle@developmentseed.org>
1 parent b8bf79d commit 08d6973

File tree

12 files changed

+1188
-930
lines changed

12 files changed

+1188
-930
lines changed

docs/api/layers/heatmap-layer.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,12 @@
11
# HeatmapLayer
22

3+
!!! warning
4+
The `HeatmapLayer` is not currently working.
5+
6+
As of Lonboard v0.10, Lonboard upgraded to version 9.0 of the underlying [deck.gl](https://deck.gl/) library. deck.gl [appears to have a bug](https://github.com/visgl/deck.gl/issues/8960#issuecomment-2284791644) with the HeatmapLayer in 9.0, that has not yet been fixed.
7+
8+
Please temporarily downgrade to Lonboard v0.9 if you would like to use the `HeatmapLayer`.
9+
310
![](../../assets/duckdb-heatmap.jpg)
411

512
> Screenshot from [DuckDB Spatial](../../examples/duckdb) example

lonboard/_layer.py

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@
1010
from __future__ import annotations
1111

1212
import sys
13+
import warnings
14+
from textwrap import dedent
1315
from typing import (
1416
TYPE_CHECKING,
1517
List,
@@ -1928,6 +1930,19 @@ class HeatmapLayer(BaseArrowLayer):
19281930
def __init__(
19291931
self, *, table: ArrowStreamExportable, **kwargs: Unpack[HeatmapLayerKwargs]
19301932
):
1933+
err_msg = """\
1934+
The `HeatmapLayer` is not currently working.
1935+
1936+
As of Lonboard v0.10, Lonboard upgraded to version 9.0 of the underlying
1937+
[deck.gl](https://deck.gl/) library. deck.gl [appears to have a
1938+
bug](https://github.com/visgl/deck.gl/issues/8960#issuecomment-2284791644) with
1939+
the HeatmapLayer in 9.0, that has not yet been fixed.
1940+
1941+
Please temporarily downgrade to Lonboard v0.9 if you would like to use the
1942+
`HeatmapLayer`.
1943+
"""
1944+
warnings.warn(dedent(err_msg), UserWarning)
1945+
19311946
# NOTE: we override the default for _rows_per_chunk because otherwise we render
19321947
# one heatmap per _chunk_ not for the entire dataset.
19331948
table_o3 = Table.from_arrow(table)

0 commit comments

Comments
 (0)