Tom Augspurger
Tom Augspurger
You're probably right about this being related to the compat level though. If we manually do the conversion and print what we get for various compat levels: ```python import cudf_polars.containers...
Phew, this probably does end up in pylibcudf's handling of validity maps on sliced columns like Lawrence guessed. https://github.com/rapidsai/cudf/issues/19159 is a clean issue with a reproducer. I'm not yet sure...
Thanks @davidwendt, I can confirm that https://github.com/rapidsai/cudf/pull/19174 fixes both https://github.com/rapidsai/cudf/issues/19159 and this issue.
Yes, thanks. We don't yet have explicit test coverage for this in cudf-polars, but that'll come as part of https://github.com/rapidsai/cudf/pull/19146 so I think we're good here.
FYI @rjzamora this will have some overlap with https://github.com/rapidsai/cudf/pull/19130. Any new keys we add to `state` will need a field in `CachingVisitorState`.
Hmm, failing on 3.10: https://github.com/rapidsai/cudf/actions/runs/15594634303/job/43923137155?pr=19135#step:10:7551 ``` │ │ TypeError: cannot inherit from both a TypedDict type and a non-TypedDict base class ``` This might need to wait until we drop...
Just seeing if CI passes with `TypedDict` from typing_extensions. The docs say this should work. If it does, we can discuss adding that as a dependency for python=3.10. Maybe not...
I'd like to add a dependency on `typing-extensions` for just python 3.10. For 3.11 and newer, we don't need typing-extensions. Here's what I think is the desired output: ```diff diff...
Oh, I hadn't considered embedding the marker I want (`python_version < '3.11`) inside the `depedencies.yaml`. That seems to work nicely, thanks.
@gforsyth when you get a chance can you verify that https://github.com/rapidsai/cudf/pull/19135/commits/2077154e5e2827c7db624022597b6be30f7a6888 looks right for the rattler-build side of things? Based on https://rattler.build/latest/selectors/#available-variables it looks OK, but I haven't used rattler-build...