[autumnust]

Summary

Ask for compiler or library baselines when reviewing new or expanded custom GPU kernels.

Description of the change

Scope the review instruction to sdm/_kernels/**. Reviewers should request a practical alternative comparison or a concrete explanation of incompatibility, check that workloads and numerical requirements match, and distinguish startup cost from prediction latency. Missing evidence is a justification question; performance should not be inferred from source code alone.

Validation

Parsed the YAML and validated the configuration against CodeRabbit's published JSON schema.

[copy-pr-bot]

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.91%. Comparing base (a7e57e2) to head (313b1c2).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1255   +/-   ##
=======================================
  Coverage   82.91%   82.91%           
=======================================
  Files         173      173           
  Lines       11191    11191           
=======================================
  Hits         9279     9279           
  Misses       1912     1912           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Source mode: Auto
• Repository: NVIDIA/structured-data-models
• Branch: codex/coderabbit-kernel-baselines
• File: .coderabbit.yaml
• Commit: 313b1c2
• Layers: Repository YAML
• Review profile: QUIET
• Plan: Enterprise
• Run ID: 4414bbd3-36bb-4504-9d8f-637a9d0c0b4c

📥 Commits

Reviewing files that changed from the base of the PR and between a7e57e2 and 313b1c2.

📒 Files selected for processing: 1

• .coderabbit.yaml

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

📝 Summary

Summary by CodeRabbit

• Chores
  • Updated internal review guidance. This update does not change any user-facing features, behavior, or performance.

The review configuration adds path-specific instructions for changes to sdm/_kernels/**. The instructions cover performance comparisons, workload and numerical requirements, startup and prediction latency, and performance evidence.

Changes: Kernel review guidance

Layer / File(s)	Summary
Custom GPU kernel review instructions .coderabbit.yaml	Adds review checks for practical alternatives, comparable performance evidence, workload and numerical requirements, and separate startup and prediction latency. It states that source code alone does not establish performance.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~4 minutes

Merge Risk: ⚪ Minimal · up to 313b1

The scoped review guidance is mergeable with no actionable risk identified.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title "Ask for compiler baselines in custom GPU kernel reviews" matches the main change, which adds a path-scoped review instruction for sdm/_kernels/** that asks reviewers to request compiler or …
Description check		The description states the change scope (sdm/_kernels/**), the requested baseline comparison or incompatibility explanation, the workload and numerical checks, the startup versus prediction latency di…
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title "Ask for compiler baselines in custom GPU kernel reviews" matches the main change, which adds a path-scoped review instruction for sdm/_kernels/** that asks reviewers to request compiler or library baseline comparisons for new or expanded custom GPU kernels. The title is concise, specific, and free of noise.

Full details: Description check

Explanation

The description states the change scope (sdm/_kernels/**), the requested baseline comparison or incompatibility explanation, the workload and numerical checks, the startup versus prediction latency distinction, and the YAML validation against the published schema. It relates directly to the changeset.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{Comment @coderabbitai help to get the list of available commands.}

[akihironitta]

👍