pFad - Phone/Frame/Anonymizer/Declutterfier! Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

URL: http://github.com/plotly/plotly.js/pull/7722

om/assets/global-0bd78641c0a1f3e0.css" /> Prevent NaN lonShift from crashing hover by platinops · Pull Request #7722 · plotly/plotly.js · GitHub
Skip to content

Prevent NaN lonShift from crashing hover#7722

Open
platinops wants to merge 1 commit intoplotly:masterfrom
platinops:scattermap-hover-lonshift-nan
Open

Prevent NaN lonShift from crashing hover#7722
platinops wants to merge 1 commit intoplotly:masterfrom
platinops:scattermap-hover-lonshift-nan

Conversation

@platinops
Copy link
Copy Markdown

@platinops platinops commented Feb 24, 2026 •
edited by emilykl
Loading

Prevent NaN lonShift from crashing hover on maps when xval is undefined.
Fixes #7702.
I created this since I saw PR #7709 was abandoned/deleted by its author.

@camdecoster
Copy link
Copy Markdown
Contributor

camdecoster commented Feb 25, 2026

Thanks for the PR. We'll take a look at follow up.

@camdecoster
Copy link
Copy Markdown
Contributor

camdecoster commented Mar 27, 2026

I think that just catching NaN values is a band-aid fix. I looked into why the values are NaN to begin with, and it has to do with the fact that xval and yval aren't getting set when Plotly.Fx.hover is called with pointNumber in the payload. Could you see if there's a way to set xval and yval to the proper value? Additionally, there might be other places where these values need to be set.

@platinops
Copy link
Copy Markdown
Author

platinops commented Apr 3, 2026

I am not a JS developer and not familiar with the plotly.js code at all, so I can only guess: Maybe xval and yval are optional since the coordinates are already part of pointData?

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

Reviewers

No reviews

Assignees

@emilykl emilykl

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG]: Plotly.Fx.hover error when used on Plotly map

3 participants

@platinops @camdecoster @emilykl
pFad - Phonifier reborn

Pfad - The Proxy pFad © 2024 Your Company Name. All rights reserved.




Check this box to remove all script contents from the fetched content.



Check this box to remove all images from the fetched content.


Check this box to remove all CSS styles from the fetched content.


Check this box to keep images inefficiently compressed and original size.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy