Skip to content

BUG: Plotly.restyle gives error if array has empty values #4031

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
mafar opened this issue Jul 11, 2019 · 1 comment · Fixed by #4038
Closed

BUG: Plotly.restyle gives error if array has empty values #4031

mafar opened this issue Jul 11, 2019 · 1 comment · Fixed by #4038
Assignees
@etpinard
Labels
bug something broken

Comments

@mafar
Copy link
Contributor

mafar commented Jul 11, 2019 •
edited
Loading

when Plotly.restyle is called with array with empty values , gives error
y: [["Morning", "Afternoon", "Evening", "", ""]]

CODE: https://codepen.io/mafar/pen/agMvEO

var data = [
  {
    z: [[1, 20, 30, 50, 1], [20, 1, 60, 80, 30], [30, 60, 1, -10, 20]],
    x: ["Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "", ""],
    y: ["Morning", "Afternoon", "Evening"],
    type: "heatmap"
  }
];

Plotly.newPlot("myDiv", data, {}, { showSendToCloud: true });
// Plotly.restyle with empty array values
var update = {
  y: [["Morning", "Afternoon", "Evening", "", ""]]
};
Plotly.restyle("myDiv", update, 0);

See attached image:
ion
@etpinard
Copy link
Contributor

Thanks for reporting!

This isn't a restyle bug it looks like:

var data = [
  {
    z: [[1, 20, 30, 50, 1], [20, 1, 60, 80, 30], [30, 60, 1, -10, 20]],
    x: ["Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "", ""],
    y: ["Morning", "Afternoon", "Evening", "", ""],
    type: "heatmap"
  }
];

Plotly.newPlot(gd, data, {}, { showSendToCloud: true });

also spits out the same exception.

@etpinard etpinard added the bug something broken label Jul 11, 2019
@etpinard etpinard mentioned this issue Jul 11, 2019
Closed
@etpinard etpinard self-assigned this Jul 11, 2019
@etpinard etpinard mentioned this issue Jul 11, 2019
Merged
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 clean2dArray for categorial x/y with extra items plotly/plotly.js
2 participants
@mafar @etpinard