SAM3: Prevent using transformers 5.10#9
Conversation
It is broken, see huggingface/transformers#46402
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe SAM3 segmentation plugin's ChangesTransformers Dependency Constraint
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
/review |
What has changed and why?
Prevent using transformers 5.10, which is broken. See huggingface/transformers#46402
How has it been tested?
Manually
Did you update Readme.md and plugins.toml?
Updated the
transformersdependency constraint inplugins/sam3_segmentation/pyproject.tomlfromtransformers>=4.57.2totransformers>=4.57.2,<5.10to prevent use of the broken v5.10 release. Added inline comments referencing the upstream Hugging Face issue for context.