Skip to content

Conversation

ldematte
Copy link
Contributor

Some additional tests to increase our test coverage.

Good to review, but we might want to expand this after #1283 (or vice-versa, whichever comes first).

Copy link

copy-pr-bot bot commented Sep 24, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@benfred benfred added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Sep 24, 2025
…pes-tests

# Conflicts:
#	java/cuvs-java/src/test/java/com/nvidia/cuvs/CagraBuildAndSearchIT.java
…pes-tests

# Conflicts:
#	java/cuvs-java/src/test/java/com/nvidia/cuvs/CagraBuildAndSearchIT.java
@cjnolet
Copy link
Member

cjnolet commented Sep 26, 2025

/ok to test 4a9f56c

@mythrocks
Copy link
Contributor

Looks like this is going to need a rebase.

@cjnolet will keep me honest: Given the code-freeze, maybe we should retarget this to 25.12.

@ldematte
Copy link
Contributor Author

ldematte commented Oct 2, 2025

@mythrocks @cjnolet I can do that, but at the moment it seems that 25.12 is behind 25.10, so you probably want to incorporate all changes from 25.10 before I rebase, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Development

Successfully merging this pull request may close these issues.

4 participants