Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove raft-ann-bench #2497

Merged
merged 9 commits into from
Nov 16, 2024
Merged

Conversation

cjnolet
Copy link
Member

@cjnolet cjnolet commented Nov 15, 2024

This PR removes raft-ann-bench from the conda packages, build system, and documentation. This removal was previously announced for the 24.12 release in #2448.

@cjnolet cjnolet added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Nov 15, 2024
@cjnolet cjnolet self-assigned this Nov 15, 2024
@cjnolet cjnolet requested review from a team as code owners November 15, 2024 17:00
@cjnolet cjnolet requested a review from raydouglass November 15, 2024 17:00
Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a few more instances of ANN_BENCH in docs/source/ and .github/workflows/pr.yaml and a few other spots.

cpp/CMakeLists.txt Outdated Show resolved Hide resolved
cpp/CMakeLists.txt Outdated Show resolved Hide resolved
build.sh Outdated Show resolved Hide resolved
@bdice bdice requested a review from a team as a code owner November 15, 2024 18:41
@github-actions github-actions bot added the ci label Nov 15, 2024
@bdice bdice changed the title 2412 remove ann bench Remove raft-ann-bench Nov 15, 2024
@bdice
Copy link
Contributor

bdice commented Nov 15, 2024

@cjnolet I think this PR should be ready once we get CI passing. It turns out there were quite a few pieces we could prune as a result of removing raft-ann-bench. I ran devcontainer builds locally and it seems fine. I see the pip devcontainer failed, I will rerun it soon. I don't think the failure is due to these changes. I think it hit OOM during the build, which makes the job show no logs.

@cjnolet
Copy link
Member Author

cjnolet commented Nov 16, 2024

/merge

@rapids-bot rapids-bot bot merged commit fccf33e into rapidsai:branch-24.12 Nov 16, 2024
70 checks passed
rapids-bot bot pushed a commit that referenced this pull request Dec 17, 2024
Proposes some small cleanup for `dependencies.yaml`

* removes `rapids_build_setuptools` dependency group
  - *#2497 removed the last use of `setuptools` here*
* breaks `cuda-python` and `rmm` out into `depends_on_*` groups to reduce duplication, and for consistency with other RAPIDS projects ([docs explaining this](https://github.com/rapidsai/build-planning/blob/d9e3c606d95c835ee384ac6480a4af0ac6cb024a/docs/docs/packaging.md#L181))
* alphabetizes lists

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Bradley Dice (https://github.com/bdice)

URL: #2529
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci CMake cpp improvement Improvement / enhancement to an existing function non-breaking Non-breaking change python
Projects
Development

Successfully merging this pull request may close these issues.

3 participants