search

rapidsai / docs

RAPIDS Documentation Site
https://docs.rapids.ai
37 stars 52 forks source link

Remove librmm docs #517

Closed raydouglass closed 5 months ago

raydouglass commented 5 months ago

librmm docs were merged into rmm docs and 24.02 was the last version to have separated librmm docs.

Now that the legacy version is 24.04, the librmm docs can be removed.

Ref: https://github.com/rapidsai/docs/pull/478 https://github.com/rapidsai/docs/actions/runs/9421083274/job/25954311807

netlify[bot] commented 5 months ago

Deploy Preview for docs-rapids-ai ready!

Built without sensitive environment variables

Name Link
Latest commit 560fba5af10300a4925b767e0a62acf65c3120bb
Latest deploy log https://app.netlify.com/sites/docs-rapids-ai/deploys/66634bb8e375450008d88cc1
Deploy Preview https://deploy-preview-517--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 site configuration.

raydouglass commented 5 months ago

We need a solution for libcudf's docs that currently depend on librmm's Doxygen docs.

After some internal discussions, this does need to be addressed, but we don't want to block updating to the 24.06 docs, so going to override and merge this PR.

bdice commented 4 months ago

Following up: https://github.com/rapidsai/rmm/pull/1604 re-adds librmm docs. From discussion with @raydouglass, we don't need to link to them publicly on docs.rapids.ai, as they are already included in the rmm docs (both C++ and Python). The librmm docs will just be used to cross-reference from libcudf docs.