Skip to content

add no_libucx matrix option in dependencies.yaml#607

Merged
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
trxcllnt:fix/dependencies-without-libucx
Mar 6, 2026
Merged

add no_libucx matrix option in dependencies.yaml#607
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
trxcllnt:fix/dependencies-without-libucx

Conversation

@trxcllnt
Copy link
Contributor

@trxcllnt trxcllnt commented Mar 5, 2026

This option allows generating dependencies without libucx in the dependencies list, which is something we have to do for NVAIE/DLFW builds.

@trxcllnt trxcllnt requested a review from a team as a code owner March 5, 2026 23:30
@trxcllnt trxcllnt requested a review from bdice March 5, 2026 23:30
@trxcllnt trxcllnt added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Mar 5, 2026
@pentschev
Copy link
Member

I'm not sure I understand this, it allows installing packages that do not contain libucx as a dependency but those packages are built externally (i.e., not available in CI here), is that it? Furthermore, I thought NVAIE only used wheels, why do we need to also add this exception to conda dependencies?

@jameslamb jameslamb self-requested a review March 6, 2026 14:49
Copy link
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

I agree with this, and having this "early return" style at the top of the dependencies groups is the clearest way to do it.

To answer your questions @pentschev ...

allows installing packages that do not contain libucx as a dependency but those packages are built externally (i.e., not available in CI here), is that it?

Yes, exactly. In NVAIE builds we prefer system-installed libraries for UCX to those from wheels.

I thought NVAIE only used wheels, why do we need to also add this exception to conda dependencies?

It does only use wheels. I'm guessing @trxcllnt updated the conda-only lists too for the benefit of the open-source RAPIDS devcontainers, which can pull in a system install of UCX via https://github.com/rapidsai/devcontainers/tree/main/features/src/ucx

@trxcllnt
Copy link
Contributor Author

trxcllnt commented Mar 6, 2026

I can delete it from the conda entry, but I thought it prudent to include for consistency. If we generate a dependency file with no_libucx=true, it's because we are using some UCX not available via conda or pip. NVAIE uses a special HPC-X build of UCX, so we can't have the libucx wheel in the env as well.

@jameslamb
Copy link
Member

I support keeping it there for conda too.

@jameslamb
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit 9f0a9dc into rapidsai:main Mar 6, 2026
163 of 165 checks passed
@trxcllnt trxcllnt deleted the fix/dependencies-without-libucx branch March 6, 2026 20:42
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants