vortex-data / vortex / 16579680125

88%

push

github

[chore] add filter conformance test & fill existing conformance testing gaps (#4041)

### Summary of Changes

1. Created Filter Conformance Test Harness

- Added /vortex-array/src/compute/conformance/filter.rs with flexible
test suite that works with arrays of any length
- Tests scenarios: all true, all false, selective filtering, single
element, nullable arrays, empty arrays, and mismatched lengths

2. Added MaskKernel Implementations

- ZigZag encoding: Implemented MaskKernel in
/encodings/zigzag/src/compute.rs
- ALP encoding: Implemented MaskKernel in
/encodings/alp/src/alp/compute/mask.rs with proper patch handling

3. Added filter and mask conformance tests to ZigZag, ALP, ALPRD,
ByteBool, DateTimeParts, DecimalByteParts, Dict, BitPacked (FastLanes),
FoR, FSST, Sparse, Sequence

4. Restructured ALP Compute Module
  - mod.rs - module declarations
  - take.rs - take operations
  - mask.rs - mask operations with MaskKernel
  - filter.rs - filter operations with tests

5. Fixed Test Coverage Gaps
- Added nullable array tests for primitive and bool arrays (both filter
and mask)
  - Added nullable chunked array test
- Fixed FSST filter test to properly create FSST arrays using
VarBinBuilder and compressor
- Fixed DateTimeParts filter test to use TemporalArray with Extension
dtype

All changes maintain structural consistency by placing filter tests in
filter.rs files and mask tests in mask.rs files across the codebase.

---------

Signed-off-by: Will Manning <will@willmanning.io>

598 of 624 new or added lines in 36 files covered. (95.83%)

44234 of 53768 relevant lines covered (82.27%)

169270.66 hits per line