-
-
Notifications
You must be signed in to change notification settings - Fork 109
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
Add Dask DataFrame plot test #1131
Conversation
LGTM! I've added It took me a while before I figured out this PR wasn't targeting main, once the branch it targets is merged I will point it to main so the commit history is as expected. |
Fine by me.
Yeah, didn't want the merge conflict, so based in on the Polars PR. This should automatic be re-targeted when Polars Pr are merged. Though, something like this could be needed |
Continue the work done in #1129 by adding simple plotting tests of
dask.DataFrame
anddask.Series
.I discovered a small bug that
kde
anddensity
did error out becausepd.melt
was used.Which means that something like this errors out on main: