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

vis_*_ly still available? #129

Open
rinzebloem opened this issue Nov 8, 2020 · 2 comments
Open

vis_*_ly still available? #129

rinzebloem opened this issue Nov 8, 2020 · 2 comments

Comments

@rinzebloem
Copy link

Are the plotly wrappers still available for the visdat package?

I am not able to use vis_miss_ly() after installing the package (both from CRAN or dev version), R simply doesn't recognize the function.

@njtierney
Copy link
Collaborator

Hi there!

To get plotly wrappers around visdat functions, you can run:

library(visdat)
library(plotly)
vis_dat(airquality)
ggplotly()

The vis_*_ly functions are in another branch and development slowed as I wasn't sure the benefit over using ggplotly() was worth the tradeoff.

Let me know if this doesn't fit your usecase and then I'll have another data point towards implementing the vis_*_ly family.

@rinzebloem
Copy link
Author

Thanks for the quick response! I understand the reasons for taking vis_*_ly out of the package.
Unfortunately, wrapping vis_miss() into ggplotly() is simply too slow to be reasonable for my dataset (time series with 17k observations of 1.5k variables) and according to this documentation, the vis_*_ly functions were supposed to be "3000 times faster". However, I understand from your comment that this does not seem to be the case?

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

No branches or pull requests

2 participants