Skip to content

Issue in Sankey Plot: Error in proposed[["hoveron"]] #1315

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

Closed
cpsievert opened this issue Aug 2, 2018 · 5 comments
Closed

Issue in Sankey Plot: Error in proposed[["hoveron"]] #1315

cpsievert opened this issue Aug 2, 2018 · 5 comments

Comments

@cpsievert
Copy link
Collaborator

From @KoningD

Example https://plot.ly/r/sankey-diagram/#basic-sankey-diagram under Add Links gives the error:

Error in proposed[["hoveron"]] : subscript out of bounds

I tried it with the following versions:
r version 3.5.1, plotly_4.8.0
r version 3.4.2, plotly_4.8.0

@lightcross
Copy link

lightcross commented Aug 3, 2018

Thank you for your work.
You said you fixed the bug, but unfortunately, the same error still appears.

I tested your code with R version 3.5.1, plotly_4.8.0.
I'm trying to draw a Sankey diagram, but I see the same type of error repeatedly.

Please help me!

@cpsievert
Copy link
Collaborator Author

I didn't say it was fixed, I just moved the issue's location

@aarora79
Copy link

I encountered the same problem when my Sankey chart stopped working after upgrade to plotly 4.8.0. Reverting to 4.7.1 fixed the problem ofcourse (devtools::install_version("plotly", "4.7.1")). I had upgraded to 4.8.0 to get a fix for the following warning as described in #985.

Warning in origRenderFunc() :
  Ignoring explicitly provided widget ID "63f851f7f801"; Shiny doesn't use them

@cpsievert
Copy link
Collaborator Author

As it turns out, the example itself has a bug...the domain should be a list of two atomic vectors (both of length 2):

domain = list(
    x =  c(0,1),
    y =  c(0,1)
 )

Not an atomic vector with four elements:

domain = c(
    x =  c(0,1),
    y =  c(0,1)
 )

If anyone sees this error due to a different problem please let me know.

@aarora79
Copy link

Verified, works with
domain = list(
x = c(0,1),
y = c(0,1)
)

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

3 participants