Skip to content

Conversation

@rhdong
Copy link
Member

@rhdong rhdong commented Feb 28, 2025

No description provided.

Add python APi and update doc for python API
@rhdong rhdong requested review from a team as code owners February 28, 2025 20:02
@rhdong rhdong added feature request New feature or request non-breaking Introduces a non-breaking change and removed cpp Python labels Feb 28, 2025
@rhdong rhdong closed this Feb 28, 2025
@rhdong rhdong reopened this Mar 1, 2025
@rhdong rhdong requested review from a team as code owners March 1, 2025 01:00
@rhdong rhdong requested a review from raydouglass March 1, 2025 01:00
@github-actions github-actions bot added the ci label Mar 1, 2025
@github-actions github-actions bot removed the ci label Mar 3, 2025
rapids-bot bot pushed a commit that referenced this pull request Mar 4, 2025
#695 introduced a conda dependency on `"sklearn"`. There is no `sklearn`... the package is called `scikit-learn`.

This fixes that.

## Notes for Reviewers

### How this fixes CI

On #738, @rhdong was facing the following problem. The `conda-cpp-build` jobs were producing packages with version `25.04.00a94`:

```text
BUILD START: ['libcuvs-25.04.00a94-cuda11_250303_g2ffe160_94.conda',
'libcuvs-static-25.04.00a94-cuda11_250303_g2ffe160_94.conda',
'libcuvs-examples-25.04.00a94-cuda11_250303_g2ffe160_94.conda',
'libcuvs-tests-25.04.00a94-cuda11_250303_g2ffe160_94.conda']
```

([logs link](https://github.com/rapidsai/cuvs/actions/runs/13636442011/job/38116419908?pr=738#step:9:489))

But at test time, they were getting `25.05.00a84`:

```text
...
libcuvs                   25.04.00a84     cuda11_250224_ga1e0cc0_84    rapidsai-nightly
```

([logs link](https://github.com/rapidsai/cuvs/actions/runs/13636442011/job/38125502858?pr=738#step:9:4583))

It looks like this `sklearn` dependency was the root cause, as explained in #743 (comment)

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

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

URL: #743
@rhdong rhdong closed this Mar 4, 2025
jiangyinzuo pushed a commit to jiangyinzuo/cuvs that referenced this pull request Mar 27, 2025
rapidsai#695 introduced a conda dependency on `"sklearn"`. There is no `sklearn`... the package is called `scikit-learn`.

This fixes that.

## Notes for Reviewers

### How this fixes CI

On rapidsai#738, @rhdong was facing the following problem. The `conda-cpp-build` jobs were producing packages with version `25.04.00a94`:

```text
BUILD START: ['libcuvs-25.04.00a94-cuda11_250303_g2ffe160_94.conda',
'libcuvs-static-25.04.00a94-cuda11_250303_g2ffe160_94.conda',
'libcuvs-examples-25.04.00a94-cuda11_250303_g2ffe160_94.conda',
'libcuvs-tests-25.04.00a94-cuda11_250303_g2ffe160_94.conda']
```

([logs link](https://github.com/rapidsai/cuvs/actions/runs/13636442011/job/38116419908?pr=738#step:9:489))

But at test time, they were getting `25.05.00a84`:

```text
...
libcuvs                   25.04.00a84     cuda11_250224_ga1e0cc0_84    rapidsai-nightly
```

([logs link](https://github.com/rapidsai/cuvs/actions/runs/13636442011/job/38125502858?pr=738#step:9:4583))

It looks like this `sklearn` dependency was the root cause, as explained in rapidsai#743 (comment)

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

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

URL: rapidsai#743
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cpp feature request New feature or request non-breaking Introduces a non-breaking change Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant