WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@gforsyth
Copy link
Contributor

  • docs: add note about usage of rapids-package-name in reproducing ci
  • docs: add example of using --noarch flag with rapids-get-pr-artifact

xref rapidai/build-planning#43

@netlify
Copy link

netlify bot commented Dec 10, 2025

Deploy Preview for docs-rapids-ai ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 551f16e
🔍 Latest deploy log https://app.netlify.com/projects/docs-rapids-ai/deploys/693c5cf7e963430008a96d00
😎 Deploy Preview https://deploy-preview-714--docs-rapids-ai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

**Note:** By default `rapids-get-pr-artifact` uses the most recent commit from the specified PR.
A commit hash from the dependent PR can be added as an optional 4th argument to pin testing to a specific commit.

**Example 3:** Integration testing `cuml` (Python) with a `noarch` build of `cuxfilter`
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's use a real example. cuML doesn't use cuxfilter. Maybe cudf using dask-cuda? Or something using dask-cudf? There might be better examples.

# This ensures that when conda packages are built with strict channel priority enabled,
# the locally-downloaded packages will be preferred to remote packages (e.g. nightlies).
#
RAPIDS_PREPENDED_CONDA_CHANNELS=("${CUXFILTERCHANNEL}")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
RAPIDS_PREPENDED_CONDA_CHANNELS=("${CUXFILTERCHANNEL}")
RAPIDS_PREPENDED_CONDA_CHANNELS=("${CUXFILTER_CHANNEL}")

@gforsyth
Copy link
Contributor Author

Updated. Do we just use the merge buttons on this repo?

@jameslamb
Copy link
Member

Updated. Do we just use the merge buttons on this repo?

we do:

auto_merger: false

And that file was added only 3 days ago (#713)... before that there wasn't ops-bot.yaml here at all.

Honestly I think we should start using /merge. I don't know of any reason not to here. I'd be happy to approve a PR changing that (or fine with you just pushing a commit here that does that).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants