Skip to content

Bar plots are drawn incorrectly when null values are present #372

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
aleburato opened this issue Mar 31, 2016 · 3 comments
Closed

Bar plots are drawn incorrectly when null values are present #372

aleburato opened this issue Mar 31, 2016 · 3 comments
Labels
bug something broken community community contribution

Comments

@aleburato
Copy link
Contributor

Hi guys, when using bar plots it seems that the bar width gets calculated based on the number of non-null Y values in the data set, while all the x values are added to the axis. This leads to a messed up display.

See: http://codepen.io/aleburato/pen/yOomYa?editors=0010

Ciao!
@aleburato
Copy link
Contributor Author

After further investigation, the problem only shows up when the first OR the second value of the dataset is null.

Pen updated.

@mdtusz
Copy link
Contributor

mdtusz commented Apr 20, 2016

Sorry we haven't triaged this until now! Definitely a bug (and a somewhat strange one as well)!

@mdtusz mdtusz added bug something broken community community contribution labels Apr 20, 2016
seibs added a commit to seibs/plotly.js that referenced this issue May 14, 2016
@seibs
Bar: fix issue plotly#372 - bar widths with nonnumeric sizes
f326975
@seibs seibs mentioned this issue May 14, 2016
Open
seibs added a commit to seibs/plotly.js that referenced this issue May 16, 2016
@seibs
Bar: fix issue plotly#372 - bar widths with nonnumeric sizes
d0a5f5e
@seibs seibs mentioned this issue May 16, 2016
Merged
seibs added a commit to seibs/plotly.js that referenced this issue May 17, 2016
@seibs
Bar: fix issue plotly#372 - bar widths with nonnumeric sizes
dd2251d
etpinard added a commit that referenced this issue May 17, 2016
@etpinard
Merge pull request #542 from seibs/fix-372
75614a5 
Bar: fix issue #372 - bar widths with nonnumeric sizes
@etpinard
Copy link
Contributor

fixed in seibs@dd2251d

@imaloserbaby imaloserbaby mentioned this issue Jul 3, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug something broken community community contribution
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mdtusz @etpinard @aleburato