Conversation
|
@claude are you here, can you do a review too? |
|
Claude finished @yiyixuxu's task in 1m 28s —— View job PR Review
All 5 changed files in this PR are outside That said, here are a few observations worth noting on the utility scripts and CI files:
Overall the PR is well-structured. The main concern worth addressing before merge is the silent JSON parse failure in |
yiyixuxu
left a comment
There was a problem hiding this comment.
this is awesome! thanks
I left one comment
| from huggingface_hub import InferenceClient | ||
|
|
||
|
|
||
| SYSTEM_PROMPT = """\ |
There was a problem hiding this comment.
I think we have a missing-info/ or missing-code-example label, cam we add it here too?
There was a problem hiding this comment.
Reusing our existing needs-code-example label for that.
https://github.com/huggingface/diffusers/pull/13380/changes#diff-ed09b43f8018cb6694ae7bc3d686d34f5267f7bba506ca6b650d1ddb2b54a7b9R36
Did you also mean missing info like env details?
There was a problem hiding this comment.
nice!
oh i just meant needs-code-example, but not sure if you want to re-use name. we can add one for missing env details too, but not as important I think!
Did you also mean missing info like env details?
|
A few suggestions:
|
yiyixuxu
left a comment
There was a problem hiding this comment.
thanks for working on this!
super excited to see it in action
| from huggingface_hub import InferenceClient | ||
|
|
||
|
|
||
| SYSTEM_PROMPT = """\ |
There was a problem hiding this comment.
nice!
oh i just meant needs-code-example, but not sure if you want to re-use name. we can add one for missing env details too, but not as important I think!
Did you also mean missing info like env details?
What does this PR do?
Addresses suggestions brought up in #13343 by adding scripts/workflows to auto labels issues and PRs .
For Issues:
Use
Qwen/Qwen3.5-35B-A3Bvia Inference Providers to parse issues titles and content and assign appropriate labels from a fixed set.For PRs:
- S (<50 LOC diff)
- M (<200 LOC diff)
- L (>200 LOC diff)
Fixes # (issue)
#13343
Before submitting
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.