Skip to content

create_hexbin_mapbox uses *_map chart types instead of *_mapbox #5358

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
emilykl merged 10 commits into from
Oct 22, 2025
+56 −37
Merged

create_hexbin_mapbox uses *_map chart types instead of *_mapbox #5358

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
7c589f8
Refactored create_hexbin_mapbox to use *_map chart types instead of *…
ajlien Oct 1, 2025
b9a271d
Updated create_hexbin_mapbox docstring
ajlien Oct 1, 2025
565a92c
Merge branch 'main' into update-hexbin-map-no-mapbox
emilykl Oct 1, 2025
cd2d4ad
Renamed create_hexbin_mapbox to create_hexbin_map
ajlien Oct 22, 2025
bd178f6
Ruff formatting
ajlien Oct 22, 2025
5b38811
Merge branch 'main' into update-hexbin-map-no-mapbox
ajlien Oct 22, 2025
dd3bd89
More ruff formatting
ajlien Oct 22, 2025
ebb6209
Merge branch 'update-hexbin-map-no-mapbox' of github.com:ajlien/plotl…
ajlien Oct 22, 2025
4d8fbc4
Updated figure factory init
ajlien Oct 22, 2025
9773b5f
Removed Mapbox-related content from hexbin examples
ajlien Oct 22, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
More ruff formatting
  • Loading branch information
@ajlien
ajlien committed Oct 22, 2025
commit dd3bd892a961af8be352c149fb9ab60d0becac6e
24 changes: 9 additions & 15 deletions plotly/figure_factory/_hexbin_map.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,9 @@ def _getBoundsZoomLevel(lon_min, lon_max, lat_min, lat_max, mapDim):
Source: https://stackoverflow.com/questions/6048975/google-maps-v3-how-to-calculate-the-zoom-level-for-a-given-bounds
"""

scale = 2 # adjustment to reflect MapBox base tiles are 512x512 vs. Google's 256x256
scale = (
2 # adjustment to reflect MapBox base tiles are 512x512 vs. Google's 256x256
)
WORLD_DIM = {"height": 256 * scale, "width": 256 * scale}
ZOOM_MAX = 18

Expand Down Expand Up @@ -293,11 +295,7 @@ def _compute_wgs84_hexbin(
{"x1": centers[:, 0], "x2": centers[:, 1]},
native_namespace=native_namespace,
)
.select(
hexagons_ids=nw.concat_str(
[nw.col("x1"), nw.col("x2")], separator=","
)
)
.select(hexagons_ids=nw.concat_str([nw.col("x1"), nw.col("x2")], separator=","))
.get_column("hexagons_ids")
)

Expand Down Expand Up @@ -425,9 +423,7 @@ def create_hexbin_map(
lon=df.get_column(args["lon"]).to_numpy(),
lat_range=lat_range,
lon_range=lon_range,
color=df.get_column(args["color"]).to_numpy()
if args["color"]
else None,
color=df.get_column(args["color"]).to_numpy() if args["color"] else None,
nx=nx_hexagon,
agg_func=agg_func,
min_count=min_count,
Expand All @@ -444,9 +440,9 @@ def create_hexbin_map(
)
)

agg_data_frame = nw.concat(
agg_data_frame_list, how="vertical"
).with_columns(color=nw.col("color").cast(nw.Int64))
agg_data_frame = nw.concat(agg_data_frame_list, how="vertical").with_columns(
color=nw.col("color").cast(nw.Int64)
)

if range_color is None:
range_color = [
Expand All @@ -460,9 +456,7 @@ def create_hexbin_map(
locations="locations",
color="color",
hover_data={"color": True, "locations": False, "frame": False},
animation_frame=(
"frame" if args["animation_frame"] is not None else None
),
animation_frame=("frame" if args["animation_frame"] is not None else None),
color_discrete_sequence=color_discrete_sequence,
color_discrete_map=color_discrete_map,
labels=labels,
Expand Down