search

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.25k stars 590 forks source link

Fix: negative index of a filtered dataframe #2163

Closed JovanVeljanoski closed 2 years ago

JovanVeljanoski commented 2 years ago

Closes https://github.com/vaexio/vaex/issues/2153

maartenbreddels commented 2 years ago

I think you didn't test with master, this was already fixed in #2129

JovanVeljanoski commented 2 years ago

I did, and i just double checked it - it fails on master, i don't know why it does not fail on CI (try in your notebooks)

maartenbreddels commented 2 years ago

i guess locally you didn't rebase, but a PR will run against the merged with master.