Skip to content

Conversation

@gouzil
Copy link
Member

@gouzil gouzil commented Aug 17, 2025

PR Category

User Experience

PR Types

Not User Facing

Description

使用性能更好,格式化效果更佳的 ruff format 替代 black 作为新的 formatter,由于我们很早就已经集成了 ruff lint,因此只是减少依赖而不会增加依赖

@paddle-bot
Copy link

paddle-bot bot commented Aug 17, 2025

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Aug 17, 2025
low=0, high=255, size=[height, width]
),
yield (
np.random.uniform(low=0, high=255, size=[height, width])
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个对吗?我咋感觉少了个逗号,这是手动改的还是自动改的

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

手动改的,改错了

格式化后:

yield (
    (
        np.random.uniform(
            low=0, high=255, size=[height, width]
        )
    ),
)

label,
True,
axis,
ignore_index, # soft_label,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
ignore_index, # soft_label,
ignore_index,

label,
True,
axis,
ignore_index, # soft_label,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
ignore_index, # soft_label,
ignore_index,

Comment on lines 239 to 242
(
1e-2,
1e-2,
),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
(
1e-2,
1e-2,
),
(1e-2, 1e-2),

@SigureMo SigureMo added HappyOpenSource 快乐开源活动issue与PR skip-ci: approval labels Aug 17, 2025
@luotao1 luotao1 merged commit adfd2d7 into PaddlePaddle:develop Aug 17, 2025
92 of 95 checks passed
@SigureMo SigureMo deleted the ruff/fmt/part-21 branch August 17, 2025 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants