From bbca7b1c9f5af8e3230c4f190c9a8d3625f5341e Mon Sep 17 00:00:00 2001 From: Kyle Edwards Date: Mon, 20 Jul 2026 15:48:24 -0400 Subject: [PATCH] Update pre-commit-hooks, fix UCXX version scheme (#3087) Update pre-commit-hooks to get a bugfix related to the `--version-scheme` argument, and use this argument in `verify-hardcoded-version-ucxx`. Authors: - Kyle Edwards (https://github.com/KyleFromNVIDIA) Approvers: - James Lamb (https://github.com/jameslamb) - Divye Gala (https://github.com/divyegala) URL: https://github.com/NVIDIA/raft/pull/3087 --- .pre-commit-config.yaml | 4 ++-- cpp/tests/stats/mean.cu | 16 +++++++++------- 2 files changed, 11 insertions(+), 9 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 29c155e383..d71bca18f1 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -85,7 +85,7 @@ repos: (?x) ^CHANGELOG[.]md$| - repo: https://github.com/rapidsai/pre-commit-hooks - rev: v1.6.0 + rev: v1.6.1 hooks: - id: verify-copyright name: verify-copyright-raft @@ -160,7 +160,7 @@ repos: [.](md|rst|svg)$ - id: verify-hardcoded-version name: verify-hardcoded-version-ucxx - args: [--fix, --version-file=UCXX_VERSION] + args: [--fix, --version-file=UCXX_VERSION, --version-scheme=semver] exclude: | (?x) (^|/)devcontainer[.]json$| diff --git a/cpp/tests/stats/mean.cu b/cpp/tests/stats/mean.cu index 0cab8ebfb4..53f7b0c469 100644 --- a/cpp/tests/stats/mean.cu +++ b/cpp/tests/stats/mean.cu @@ -174,13 +174,15 @@ const std::vector> inputshf = { {0.15f, -1.f, 1024, 64, true, 1234ULL}, {0.0001f, 0.1f, 1 << 27, 2, false, 1234ULL, 0.0001f}}; -const std::vector> inputsi8h = {{0.95f, -5, 8096, 32, false, 1234ULL, 1}, - {0.5f, 1, 8096, 10, false, 1234ULL, 10}, - {0.15f, 0, 60000, 128, false, 1234ULL, 6}, - {0.5f, -1, 8096, 256, false, 1234ULL, 2}, - {1.0f, 8, 2000, 32, true, 1234ULL, 1}, - {0.50f, -1, 20000, 64, true, 1234ULL, 5}, - {1.0f, 6, 10024, 2, false, 1234ULL, 10}}; +const std::vector> inputsi8h = { + {0.95f, -5, 8096, 32, false, 1234ULL, 1}, + {0.5f, 1, 8096, 10, false, 1234ULL, 10}, + {0.15f, 0, 60000, 128, false, 1234ULL, 6}, + {0.5f, -1, 8096, 256, false, 1234ULL, 2}, + {1.0f, 8, 2000, 32, true, 1234ULL, 1}, + // rapids-pre-commit-hooks: disable-next-line[verify-hardcoded-version] + {0.50f, -1, 20000, 64, true, 1234ULL, 5}, + {1.0f, 6, 10024, 2, false, 1234ULL, 10}}; typedef MeanTest MeanTestF; TEST_P(MeanTestF, Result)