Skip to content

Update kepler-jupyter to use kepler.gl v3.2.0 - #3219

Merged
igorDykhta merged 4 commits into
keplergl:masterfrom
karosc:update-jupyter-plugin
Mar 23, 2026
Merged

Update kepler-jupyter to use kepler.gl v3.2.0#3219
igorDykhta merged 4 commits into
keplergl:masterfrom
karosc:update-jupyter-plugin

Conversation

@karosc

@karosc karosc commented Oct 12, 2025

Copy link
Copy Markdown
Contributor

I'm not a pro js developer, but I use the keplergl python library for rendering GIS maps and wanted to fix #3045. I also tried updating the build process to accommodate @kepler.gl v3.2.0, which largely involved updating webpack to v5. I'm fine with closing this out if ya'll are not interested or think I took the wrong approach in any way.

Copilot AI review requested due to automatic review settings October 12, 2025 20:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Upgrade kepler.gl Jupyter bindings to kepler.gl v3.2.0 and migrate the build to webpack 5, while ensuring the map resizes correctly after mounting.

  • Migrate webpack config to v5 (library/type format, fallbacks, loaders) and add necessary polyfills/loaders
  • Bump kepler.gl and loaders.gl packages to 3.2.0/4.3.x respectively
  • Defer data/config injection until after mount and trigger initial resize to fix map sizing in Jupyter

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
bindings/kepler.gl-jupyter/js/webpack/config.js Webpack 5 migration across bundles, new loaders, env/provide plugins, AMD library output changes
bindings/kepler.gl-jupyter/js/webpack/build-html.js Webpack 5 output format, resolve fallbacks (fs/process), UMD library config
bindings/kepler.gl-jupyter/js/package.json Upgrade webpack/dev tooling and kepler.gl/loaders.gl versions; add process polyfill dependency
bindings/kepler.gl-jupyter/js/lib/keplergl/main.js Render callback to load data/config after mount
bindings/kepler.gl-jupyter/js/lib/keplergl/components/root.js Support onMounted callback via useEffect
bindings/kepler.gl-jupyter/js/lib/keplergl/components/app.js Trigger an initial resize on mount

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread bindings/kepler.gl-jupyter/js/webpack/config.js
Comment thread bindings/kepler.gl-jupyter/js/webpack/config.js
Comment thread bindings/kepler.gl-jupyter/js/webpack/config.js
Comment thread bindings/kepler.gl-jupyter/js/webpack/config.js
Comment thread bindings/kepler.gl-jupyter/js/webpack/config.js
Comment thread bindings/kepler.gl-jupyter/js/webpack/config.js
- update webpack to v5 and migrate configs

Signed-off-by: karosc <ckaros@outlook.com>
Signed-off-by: karosc <ckaros@outlook.com>
Signed-off-by: karosc <ckaros@outlook.com>
@karosc
karosc force-pushed the update-jupyter-plugin branch from e493203 to 5016513 Compare October 12, 2025 20:50
@igorDykhta
igorDykhta self-requested a review October 12, 2025 21:22
@karosc karosc changed the title - update kepler to v3.2.0 Update kepler-jupyter to use kepler.gl v3.2.0 Oct 12, 2025
@igorDykhta
igorDykhta requested a review from lixun910 October 27, 2025 18:39

@lixun910 lixun910 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the late reply. Thank you,, @karosc !

@karosc

karosc commented Mar 23, 2026

Copy link
Copy Markdown
Contributor Author

thanks @lixun910, who merges this? Do we just need to wait untit @igorDykhta has time? I don't want to rush things, I only want to makes sure this doesn't get forgotten.

@igorDykhta
igorDykhta merged commit bc59e88 into keplergl:master Mar 23, 2026
7 checks passed
@igorDykhta

Copy link
Copy Markdown
Collaborator

Thanks @karosc. Sorry, I’m not working on the Jupyter part, so I expected Xun to handle this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Map not shown full height and width in notebook and html embeddings, unless re-framing the window manually

4 participants