Update README for NVIDIA cuML branding - #8464
Conversation
jcrist
left a comment
jcrist
left a comment
There was a problem hiding this comment.
Approving, with a few small questions/nits. Feel free to address as you see fit. The only real big one is the one around the hardcoded versions in the links. I don't want to have to update these with each release.
| Supported operations run on the GPU. When an estimator or configuration cannot | ||
| be accelerated, `cuml.accel` uses the original CPU implementation so the rest | ||
| of the workflow can continue. See the [`cuml.accel` compatibility | ||
| documentation](https://docs.nvidia.com/cuml/26.08/cuml-accel/compatibility/) for |
There was a problem hiding this comment.
Is there a way to avoid hardcoding a versioned link here? It'd be nice to not have to update these with each release. Same for all other links.
There was a problem hiding this comment.
Right now, I don't see a way to provide these kind of deep links unversioned. I suggest we keep this as-is for now and then update this once those are available.
|
Admin-merging this as an isolated README edit that doesn't require explicit testing while we're bootstrapping after the repo migration. |
3 participants
Updates the top-level
README.mdto use NVIDIA cuML and CUDA-X Data Science branding, and presentscuml.accelalongside the GPU-native API. Replaces the manually maintained algorithm and Dask sections with links to the versioned documentation, and removes the obsolete RAPIDS branding assets.