Skip to content

Heatmap crashes with d3 errors when plotting with x-axis and y-axis traces of single element arrays #605

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

Jump to bottom
Closed
mcin-armintaheri-archive opened this issue Jun 6, 2016 · 1 comment · Fixed by #651
Closed

Heatmap crashes with d3 errors when plotting with x-axis and y-axis traces of single element arrays #605

mcin-armintaheri-archive opened this issue Jun 6, 2016 · 1 comment · Fixed by #651
Assignees
@etpinard
Labels
bug something broken

Comments

@mcin-armintaheri-archive
Copy link

http://codepen.io/armintaheri/pen/JKYGrK?editors=1010

If a given axis of a heatmap only has one element in the trace, the plot crashes and the console is filled with d3.js exceptions:

attribute height: Expected length, "NaN".
attribute width: Expected length, "NaN".
attribute x: Expected length, "NaN".
attribute y: Expected length, "NaN".
@mcin-armintaheri-archive mcin-armintaheri-archive changed the title Heatmap crashes with svg errors when plotting with x-axis and y-axis traces of single element arrays Heatmap crashes with d3 errors when plotting with x-axis and y-axis traces of single element arrays Jun 6, 2016
@etpinard
Copy link
Contributor

etpinard commented Jun 7, 2016

Yikes. Thanks for the report!

@etpinard etpinard added the bug something broken label Jun 7, 2016
@etpinard etpinard self-assigned this Jun 7, 2016
etpinard added a commit that referenced this issue Jun 15, 2016
@etpinard
make x/y of length 0 or 1 generate correct bricks [fixes #605]
b913a5b
@etpinard etpinard mentioned this issue Jun 15, 2016
Merged
@wobenshain wobenshain mentioned this issue Jul 26, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@etpinard etpinard

Labels
bug something broken
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix heatmap brick generation edge cases plotly/plotly.js
2 participants
@etpinard @mcin-armintaheri-archive