GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Update plotly.js to `v4.0.0-rc.0` by emilykl · Pull Request #5673 · plotly/plotly.py

Conversation

@emilykl

@emilykl emilykl commented Jul 22, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Description of change

  • Update plotly.js to v4.0.0-rc.0
  • Run codegen and regenerate Jupyter js bundles
  • Remove all references to mapbox in Plotly Express
  • Update tests to remove mapbox references

Note: The failing plotly.js dev build CI job can be ignored. That job runs the tests after pulling dist/plotly.js from the plotly.js master branch, so naturally it will fail since the plotly.js v4.0 changes are not yet on master.

The failing docs build can also be ignored; it will fail until the docs are updated to remove mapbox code, which will be in a separate PR.

Notes

Closes #4824 due to the color library update in plotly.js v4.0.0.
Closes #5081 by removing all Mapbox references.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
emilykl added 5 commits July 22, 2026 16:09
@emilykl
update plotly.js version to v4.0.0-rc.0 and run codegen
8bcb5e0
@emilykl
rebuild FigureWidget and Jupyter extension
9419595
@emilykl
remove all references to mapbox subplots and trace types in manually-…
52bbe59 
…edited files
@emilykl
remove mapbox references from templates
72286db
@emilykl
update tests to remove mapbox references (change to map where relevant)
960ffa9
@emilykl
emilykl force-pushed the update-plotlyjs-v4.0.0-rc.0 branch from e9e1d18 to d3c2587 Compare July 23, 2026 16:08
@emilykl
emilykl requested a review from camdecoster July 23, 2026 16:28
camdecoster
camdecoster reviewed Jul 23, 2026
View reviewed changes

@camdecoster camdecoster left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

  • Could we update the docs separately to keep things tidy?
  • I think we can remove the Stream deprecation warning in _deprecations.py
  • I still see a lot of references to linkText/link_text and showLink/show_link. Could those be removed?
  • ColorValidator will need to be updated per the v4 changes

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread _plotly_utils/basevalidators.py

@camdecoster camdecoster Jul 23, 2026

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

ColorValidator will need to be updated to remove the references and regex checks for hsv since that got removed. The regex patterns will also need to be updated to match the new color specifier support.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
emilykl reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@emilykl

emilykl commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author
  • Could we update the docs separately to keep things tidy?

Yes, as long as you're OK with the docs build failing on this PR. I'll split the docs changes into a separate PR.

  • I think we can remove the Stream deprecation warning in _deprecations.py
  • I still see a lot of references to linkText/link_text and showLink/show_link. Could those be removed?
  • ColorValidator will need to be updated per the v4 changes

👍 will do, thanks
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
emilykl added 4 commits July 23, 2026 16:46
@emilykl
remove mapbox references from codegen and rerun codegen
38988a0
@emilykl
delete removed config keys from offline.py
fa2b7da
@emilykl
mapbox -> map in figure factory README
3b1b7e0
@emilykl
remove SrcValidator
3a76e7d
@emilykl
emilykl force-pushed the update-plotlyjs-v4.0.0-rc.0 branch from cb56252 to 3a76e7d Compare July 23, 2026 20:48
emilykl added 3 commits July 23, 2026 17:16
@emilykl
remove references to Stream
a293de3
@emilykl
remove references to showLink and linkText
ed0966c
@emilykl
update ColorValidator for new color processing in plotly.js 4.0
49f3825
@emilykl
emilykl force-pushed the update-plotlyjs-v4.0.0-rc.0 branch from 141a7a8 to 49f3825 Compare July 24, 2026 17:29
emilykl
emilykl commented Jul 24, 2026
View reviewed changes
Comment thread tests/test_plotly_utils/validators/test_color_validator.py
@@ -28,26 +28,58 @@ def validator_aok_colorscale():
)


VALID_COLORS = [

@emilykl emilykl Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

@camdecoster I tried to be pretty comprehensive here, but please suggest any formats I've missed. Likewise for the invalid colors list.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@emilykl
emilykl requested a review from camdecoster July 24, 2026 17:37
camdecoster
camdecoster approved these changes Jul 24, 2026
View reviewed changes

@camdecoster camdecoster left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Looks good!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@emilykl
emilykl merged commit baf6d33 into v7.0 Jul 27, 2026
26 of 37 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@emilykl
emilykl deleted the update-plotlyjs-v4.0.0-rc.0 branch July 27, 2026 15:15
This was referenced Jul 27, 2026
Remove Mapbox references from docs #5674
Merged
Plotly doesn't recognise modern RGB syntax (+ other unexpected behaviour) #4824
Closed
Run codegen to update plotly/graph_objs/ after color docstring was updated #5679
Merged
[chore] Add CI step to check that Python codegen files are up to date #5680
Closed
@emilykl emilykl mentioned this pull request Aug 9, 2026
Fully remove all mapbox references (next major release) #5081
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@camdecoster camdecoster camdecoster approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@emilykl @camdecoster
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.