fix: Explicitly specify empty_as_null for explode#356
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes a nightly-CI breakage in the experimental schema inference code by making Polars Series.explode behavior explicit during nested-type (List/Array) dtype argument inference.
Changes:
- Explicitly pass
empty_as_null=Falsetoseries.explode(...)when inferring inner dtypes forpl.List. - Explicitly pass
empty_as_null=Falsetoseries.explode(...)when inferring inner dtypes forpl.Array(keepingshape=...handling intact).
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #356 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 56 56
Lines 3458 3463 +5
=========================================
+ Hits 3458 3463 +5 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Ah |
|
Would be a shame to give up compat with older versions over this. I feel like we just have to bite the bullet and pass / not pass the argument based on |
Motivation
This fixes nightly CI: https://github.com/Quantco/dataframely/actions/runs/28209348571/job/83566986568.