-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CodeStyle][Ruff][BUAA][G-[59-67]] Fix ruff RUF005 diagnostic for 9 files in python/paddle
#67282
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
python/paddle
python/paddle
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个 PR 和 #67261 有啥区别?都写着改了 9 个文件,但实际上都改了 18 个文件,怎么都包含了另一个?需要回退另一个,每个 PR 只包含自己的 9 个
@@ -233,7 +234,8 @@ def __init__( | |||
) | |||
|
|||
# the sparse conv restricts the shape is [H, W, in_channels, out_channels] | |||
filter_shape = self._kernel_size + [ | |||
filter_shape = [ | |||
self._kernel_size, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
明显的错误,怎么不带 *
呢?
python/paddle
python/paddle
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…iles in `python/paddle` (PaddlePaddle#67282)
PR Category
User Experience
PR Types
Devs
Description
修复如下文件 RUF005 的 Ruff 规则:
Related links
@gouzil