vaexio / vaex

Out-of-Core hybrid Apache Arrow/NumPy DataFrame for Python, ML, visualization and exploration of big tabular data at a billion rows per second 🚀
https://vaex.io
MIT License
8.28k stars 590 forks source link

Regression test for interchanging categorical columns #2135

Closed honno closed 2 years ago

honno commented 2 years ago

2122 ended up fixing a somewhat miscellaneous bug, which this PR adds a test for to prevent regressions. The bug in question was due to arrow arrays being passed around to _VaexBuffer, which isn't supported, where the fix ended up converting arrow arrays internally before initialising _VaexBuffer.

maartenbreddels commented 2 years ago

Thanks, really appriciate these PR's! Sorry that our CI is a bit flaky. We cannot give you enough green, unfortunately.