vaex
vaex copied to clipboard
Add a `__dataframe__` method to `_VaexDataFrame`
This will align the implementation with those in other libraries, xref https://github.com/data-apis/dataframe-api/issues/80.
Cc @maartenbreddels, @honno
(apologies for iterating in CI, I was a bit too lazy to set up a dev environment for these few lines of code)
This seems happy now, the 4 CI failures I inspected are all HTTPClientError: HTTP 502: Bad Gateway
.
FWIW dataframe_protocol_tests.py
and the relevant dunder-df tests in dataframe-interchange-tests
pass for this PR locally.
(If the bad gateway stuff gets resolved, note there's other issues with interchange stuff on win/mac CI https://github.com/vaexio/vaex/pull/2150#issuecomment-1237847383)
@maartenbreddels I somehow keep getting notifications for new failures on reruns of CI jobs. No worries if you are doing that on purpose - if not, something is probably off in the GitHub Actions config in this repo.
The only failures are still (7x):
FAILED tests/viz_test.py::test_heatmap_with_what[webserver_fastapi-ds_filtered-tornado_client]
Yeah, that was me :) I had to rebase, hopefully it is green now.
green enough, thanks Ralf!
great, thanks!