make 'numpydoc' dependency conda-only - #8556
Conversation
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
|
/ok to test |
|
I think this is working and ready for review. In the wheel tests, I do not see cuML's tests scripts don't print a summary of The only failing job here is
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (3)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe changes scope ChangesNumpydoc dependency scoping
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This change removes numpydoc from shared pip-based test and documentation dependencies while preserving docstring validation in Conda environments. Wheel-based environments avoid the dependency without affecting runtime behavior. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Closes #7710
As that issue describes,
numpydocis a fairly heavy dependency (it pulls in a lot of other packages). This proposes making the tests that depend on it conda-only, so wheel-based CI jobs don't end up installing it.Should help a bit with wheel-testing CI time here, but my main motivation is for other builds for RAPIDS like the devcontainers and DLFW builds, which are
pip-based.