16728097825

Committed 04 Aug 2025 04:00PM UTC coverage: 48.355% (-35.1%) from 83.429%

Build # 16728097825

Build Type

Pull #4108

github

Committed by

web-flow

Commit Message

Merge 1b2d27fd8 into 649ba9576

Pull Request Pull Request #4108: perf[vortex-array]: use all_valid instead of `invalid_count() == 0`

Run Details

1 of 1 new or added line in 1 file covered. (100.0%)

11596 existing lines in 378 files now uncovered.

18635 of 38538 relevant lines covered (48.35%)

151786.4 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/vortex-array/src/arrays/constant/compute/invert.rs

// SPDX-License-Identifier: Apache-2.0
// SPDX-FileCopyrightText: Copyright the Vortex contributors

use vortex_error::VortexResult;
use vortex_scalar::Scalar;

use crate::arrays::{ConstantArray, ConstantVTable};
use crate::compute::{InvertKernel, InvertKernelAdapter};
use crate::{ArrayRef, IntoArray, register_kernel};

impl InvertKernel for ConstantVTable {
    fn invert(&self, array: &ConstantArray) -> VortexResult<ArrayRef> {
        match array.scalar().as_bool().value() {
            None => Ok(array.to_array()),
            Some(b) => Ok(ConstantArray::new(
                Scalar::bool(!b, array.dtype().nullability()),
                array.len(),
            )
            .into_array()),
        }
    }
}

register_kernel!(InvertKernelAdapter(ConstantVTable).lift());

#[cfg(test)]
mod tests {
    use vortex_dtype::Nullability::Nullable;
    use vortex_scalar::Scalar;

    use crate::Array;
    use crate::arrays::ConstantArray;
    use crate::compute::invert;

    #[test]
    fn invert_nullable_const() {
        let constant = ConstantArray::new(Scalar::bool(false, Nullable), 10);

        let inverted = invert(constant.as_ref()).unwrap();
        assert_eq!(inverted.dtype(), constant.dtype());

        let orig = invert(inverted.as_ref()).unwrap();

        assert_eq!(orig.dtype(), constant.dtype());
        assert_eq!(orig.as_constant(), constant.as_constant())
    }
}

1	// SPDX-License-Identifier: Apache-2.0
2	// SPDX-FileCopyrightText: Copyright the Vortex contributors
3
4	use vortex_error::VortexResult;
5	use vortex_scalar::Scalar;
6
7	use crate::arrays::{ConstantArray, ConstantVTable};
8	use crate::compute::{InvertKernel, InvertKernelAdapter};
9	use crate::{ArrayRef, IntoArray, register_kernel};
10
11	impl InvertKernel for ConstantVTable {
UNCOV 12	fn invert(&self, array: &ConstantArray) -> VortexResult<ArrayRef> {	×
UNCOV 13	match array.scalar().as_bool().value() {	×
14	None => Ok(array.to_array()),	×
UNCOV 15	Some(b) => Ok(ConstantArray::new(	×
UNCOV 16	Scalar::bool(!b, array.dtype().nullability()),	×
UNCOV 17	array.len(),	×
UNCOV 18	)	×
UNCOV 19	.into_array()),	×
20	}
UNCOV 21	}	×
22	}
23
24	register_kernel!(InvertKernelAdapter(ConstantVTable).lift());
25
26	#[cfg(test)]
27	mod tests {
28	use vortex_dtype::Nullability::Nullable;
29	use vortex_scalar::Scalar;
30
31	use crate::Array;
32	use crate::arrays::ConstantArray;
33	use crate::compute::invert;
34
35	#[test]
36	fn invert_nullable_const() {
37	let constant = ConstantArray::new(Scalar::bool(false, Nullable), 10);
38
39	let inverted = invert(constant.as_ref()).unwrap();
40	assert_eq!(inverted.dtype(), constant.dtype());
41
42	let orig = invert(inverted.as_ref()).unwrap();
43
44	assert_eq!(orig.dtype(), constant.dtype());
45	assert_eq!(orig.as_constant(), constant.as_constant())
46	}
47	}

vortex-data / vortex / 16728097825

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous