Remove examples/diffusers/eval example#1694
Conversation
The diffusers image-quality eval scripts (ImageReward/CLIP-IQA/CLIP) are no longer maintained. Delete examples/diffusers/eval/ and drop the now-dangling "Evaluate Accuracy" references from examples/diffusers/README.md. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Jingyu Xin <jingyux@nvidia.com>
|
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 (6)
💤 Files with no reviewable changes (5)
📝 WalkthroughWalkthroughThe Diffusers example documentation is updated to remove evaluation feature references. The README intro sentence, navigation table, and installation guidance are modified to eliminate mentions of evaluation scripts and metrics. The full "Evaluate Accuracy" section is then deleted entirely from the document. ChangesDiffusers evaluation documentation removal
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1694 +/- ##
==========================================
+ Coverage 67.72% 68.24% +0.52%
==========================================
Files 511 511
Lines 56168 56168
==========================================
+ Hits 38037 38333 +296
+ Misses 18131 17835 -296
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
/ok to test f352498 |
|
@jingyu-ml please update Changelog to mention removal of this example |
What does this PR do?
Type of change: Documentation / example removal
Removes the unmaintained
examples/diffusers/eval/image-quality evaluation example (ImageReward / CLIP-IQA / CLIP metrics), which is no longer maintained. Also cleans up the now-dangling references to it inexamples/diffusers/README.md:requirements.txtnote (example updated fromevaltofastgen, distillation),Scope is intentionally limited to the diffusers example. The unrelated
examples/llm_ptq"Evaluate Accuracy" section is not touched.Usage
N/A — this removes example scripts; there are no library API changes.
Testing
N/A — deletion of example scripts plus documentation cleanup. Verified that
examples/diffusers/README.mdhas no remaining references to the deletedeval/directory, no broken section anchors, no trailing whitespace, and the final newline is intact.Before your PR is "Ready for review"
examples/diffusers/eval/; no publicmodeloptAPI is affected.CONTRIBUTING.md: N/A — no code copied; this removes the example's ownrequirements.txt.Additional Information
Deletes
examples/diffusers/eval/(5 files) and trims its references from the diffusers README.🤖 Generated with Claude Code
Summary by CodeRabbit
Release Notes
Documentation
Chores