vortex icon indicating copy to clipboard operation
vortex copied to clipboard

Compute function test harness

Open gatesn opened this issue 1 year ago • 2 comments

We should write a test harness for each compute function. It should ask the encoding for arrays of various dtype, length, nullability etc.

This code can be feature flagged behind a custom test-harness feature, for which there's a funky workaround to allow the same crate to enable its own optional feature as a dev-dependency. See https://users.rust-lang.org/t/how-to-reference-the-test-of-another-crate-in-the-current-crate/91965/4

gatesn avatar Dec 18 '24 18:12 gatesn

Basic setup is done, we lack coverage amongst all compute functions

gatesn avatar Apr 14 '25 08:04 gatesn

We have coverage for search_sorted and binary_numeric fwiw.

robert3005 avatar Apr 14 '25 08:04 robert3005

Closing as stale, each compute function should define a canonical implementation alongside itself (https://github.com/vortex-data/vortex/issues/3454). That means it should be relatively easy to compare results if we also have #1424

gatesn avatar Jun 04 '25 17:06 gatesn