Skip to content

remove unused intersphinx configs#94

Merged
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
jameslamb:docs/update-intersphinx
Mar 31, 2026
Merged

remove unused intersphinx configs#94
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
jameslamb:docs/update-intersphinx

Conversation

@jameslamb
Copy link
Copy Markdown
Member

Yesterday's nightly build on main failed like this:

loading intersphinx inventory 'scipy' from https://docs.scipy.org/doc/scipy/objects.inv ...
loading intersphinx inventory 'sklearn' from https://scikit-learn.org/stable/objects.inv ...
loading intersphinx inventory 'treelite' from https://treelite.readthedocs.io/en/latest/objects.inv ...
WARNING: failed to reach any of the inventories with the following issues:
intersphinx inventory 'https://docs.scipy.org/doc/scipy/objects.inv' not fetchable due to <class 'requests.exceptions.ConnectTimeout'>: HTTPSConnectionPool(host='docs.scipy.org', port=443): Max retries exceeded with url: /doc/scipy/objects.inv (Caused by ConnectTimeoutError(<HTTPSConnection(host='docs.scipy.org', port=443) at 0x7a5853ca11d0>, 'Connection to docs.scipy.org timed out. (connect timeout=None)'))
building [mo]: targets for 0 po files that are out of date

(build link)

Because of docs-hosting issues with scipy:

This PR removes intersphinx configuration for cudf and scipy... nvForest doesn't use either, and removing those reduces the risk of similar failures in the future.

Similar to rapidsai/cuml#7935

@jameslamb jameslamb requested a review from a team as a code owner March 31, 2026 18:28
@jameslamb jameslamb requested a review from msarahan March 31, 2026 18:28
@jameslamb jameslamb added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Mar 31, 2026
Copy link
Copy Markdown
Member

@jcrist jcrist left a comment

Choose a reason for hiding this comment

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

:shipit:

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ff383372-f02c-4454-b28f-cb356c3c86db

📥 Commits

Reviewing files that changed from the base of the PR and between c246e07 and dcb02b7.

📒 Files selected for processing (1)
  • docs/source/conf.py
💤 Files with no reviewable changes (1)
  • docs/source/conf.py

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Updated documentation configuration by removing external library cross-reference mappings for cuDF and SciPy.

Walkthrough

Two entries ("cudf" and "scipy") were removed from the intersphinx_mapping dictionary in the Sphinx configuration file. The remaining mappings for "numpy", "python", "sklearn", and "treelite" remain unchanged.

Changes

Cohort / File(s) Summary
Sphinx Configuration
docs/source/conf.py
Removed "cudf" and "scipy" entries from intersphinx_mapping dictionary.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • hcho3
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: removing unused intersphinx configurations from the documentation setup.
Description check ✅ Passed The description is directly related to the changeset, explaining why the intersphinx configs for cudf and scipy were removed due to build failures and docs-hosting issues.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@jameslamb
Copy link
Copy Markdown
Member Author

/merge

@rapids-bot rapids-bot bot merged commit 4c026b9 into rapidsai:main Mar 31, 2026
41 checks passed
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