Add GitHub issue template for numerical instability reports#982
Add GitHub issue template for numerical instability reports#982anandhkb wants to merge 3 commits intoNVIDIA:release/26.04from
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughAdded a new GitHub issue template for numerical instability reports and updated the bug report template to reference RAPIDS issue guidelines, advise searching existing issues, and encourage choosing the appropriate cuOpt issue template. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/ISSUE_TEMPLATE/numerical_instability.md:
- Line 14: Update the Markdown line containing the minimal bug report guide so
the URL uses HTTPS instead of HTTP; locate the string "Follow this guide
http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports" and replace
the URL with
"https://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports" to ensure
the link is secure.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 1526104c-b2b9-461d-8a9e-3dd47cad7a57
📒 Files selected for processing (1)
.github/ISSUE_TEMPLATE/numerical_instability.md
Rename numerical instability to numerical issue
|
@chris-maes and @Iroy30 for re-review, thanks! |
Currently, only have a BROAD category called
BUGwhich intuitively connotes a functional issue such as a crash, inconsistency etc.Adding a new category called
numerical instabilityto track numerical issues.