Skip to content
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.

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

Specifying bins as arrays when using group and overlay with Table.hist and Table.interactive_plots() causes an error #496

Closed
surajrampure opened this issue Apr 2, 2021 · 7 comments

Comments

@surajrampure
Copy link

Sorry for the wordy title. I've attached a screenshot of the issue.

The error doesn't occur if you remove overlay = False, or if you specify bins to be an integer rather than an array. (bins as an array works in all other cases.)

Screen Shot 2021-04-01 at 11 42 13 PM

@adnanhemani
Copy link
Member

As this is a bug with the interactive plots part of the library, @chrispyles and @khsu2000 are the point of contacts on this.

@khsu2000
Copy link
Contributor

khsu2000 commented Apr 5, 2021

Looking into this now.

@surajrampure
Copy link
Author

surajrampure commented Apr 5, 2021 via email

@khsu2000
Copy link
Contributor

khsu2000 commented Apr 5, 2021

I believe the PR I made (#480) that was merged in Feb resolves this issue. When I install the most recent version of datascience and try the command, I'm able to produce the expected graphs:
image
I also verified that using datascience before this PR was merged results in the error that you saw. Using the most up-to-date version of datascience should fix your issue.

@khsu2000
Copy link
Contributor

khsu2000 commented Apr 5, 2021

I'm unsure of if there's a more convenient location to list issues, opening new issues should be fine.

@davidwagner
Copy link
Member

@surajrampure , opening additional issues here for any other plotly integration issues you find would be great! Thank you.

@adnanhemani
Copy link
Member

Noting that this issue should now be resolved, I'm going to close this issue. @surajrampure please feel free to open new issues for the plotly integration! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants