QA: applied ruff rule PLW1514 - #48990
tarekziade wants to merge 1 commit into
Conversation
|
[For maintainers] Suggested jobs to run (before merge) run-slow: audio_spectrogram_transformer, auto, bark, beit, bertweet, biogpt, bit, blt, canary, chameleon, conditional_detr, convnext, convnextv2, cvt, d_fine, dab_detr |
|
run-slow: audio_spectrogram_transformer, auto, bark, beit, bertweet, biogpt, bit, blt, canary, chameleon, conditional_detr, convnext, convnextv2, cvt, d_fine, dab_detr |
AMD CIThis comment contains models: ["models/audio_spectrogram_transformer", "models/auto", "models/bark", "models/beit", "models/bertweet", "models/biogpt", "models/bit", "models/blt", "models/canary", "models/chameleon", "models/conditional_detr", "models/convnext", "models/convnextv2", "models/cvt", "models/d_fine", "models/dab_detr"] |
Nvidia CIThis comment contains models: ["models/audio_spectrogram_transformer", "models/auto", "models/bark", "models/beit", "models/bertweet", "models/biogpt", "models/bit", "models/blt", "models/canary", "models/chameleon", "models/conditional_detr", "models/convnext", "models/convnextv2", "models/cvt", "models/d_fine", "models/dab_detr"] |
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
Rocketknight1
left a comment
Rocketknight1
left a comment
There was a problem hiding this comment.
Yes, LGTM! I see this causing problems for Windows/non-Latin users, and the fix is pretty simple. It becomes redundant after Py3.15, but that's a few years away yet 😅
CI recapDashboard: View test results in Grafana |
|
the |
3 participants
What does this PR do?
Per #48948 (comment) added PLW1514
This means enabling
previewmode and fixing a few extra spots that were triggered by the preview mode (even withexplicit-preview-rules)most fixes here were produced by Ruff's autofix