Skip to content
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

[Typing][PEP585 Upgrade][BUAA][131-150] Use standard collections for type hints for 19 files in test/ #67229

Merged
merged 6 commits into from
Aug 10, 2024

Conversation

Whsjrczr
Copy link
Contributor

@Whsjrczr Whsjrczr commented Aug 8, 2024

PR Category

User Experience

PR Types

Others

Description

修改了以下19个文件:
modified: test/auto_parallel/hybrid_strategy/semi_auto_parallel_llama_model.py
modified: test/deprecated/legacy_test/auto_parallel_op_test.py
modified: test/dygraph_to_static/test_dynamic_shape_infermeta.py
modified: test/dygraph_to_static/test_typehint.py
modified: test/dygraph_to_static/test_typing.py
modified: test/ipu/op_test_ipu.py
modified: test/ir/inference/auto_scan_test.py
modified: test/ir/inference/program_config.py
modified: test/ir/inference/test_trt_convert_activation.py
modified: test/ir/inference/test_trt_convert_affine_channel.py
modified: test/ir/inference/test_trt_convert_anchor_generator.py
modified: test/ir/inference/test_trt_convert_arg_max.py
modified: test/ir/inference/test_trt_convert_arg_min.py
modified: test/ir/inference/test_trt_convert_argsort.py
modified: test/ir/inference/test_trt_convert_assign.py
modified: test/ir/inference/test_trt_convert_batch_norm.py
modified: test/ir/inference/test_trt_convert_bilinear_interp_v2.py
modified: test/ir/inference/test_trt_convert_bitwise_and.py
modified: test/ir/inference/test_trt_convert_bitwise_not.py
注:test/dygraph_to_static/check_approval.py报错为U036

Copy link

paddle-bot bot commented Aug 8, 2024

你的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.

@Whsjrczr Whsjrczr closed this Aug 8, 2024
@Whsjrczr Whsjrczr reopened this Aug 8, 2024
@Whsjrczr Whsjrczr changed the title [Typing][PEP585 Upgrade][BUAA][141-150] Use standard collections for type hints for 10 files in test/ir/inference/ [Typing][PEP585 Upgrade][BUAA][131-150] Use standard collections for type hints for 19 files in test/ Aug 8, 2024
@Whsjrczr Whsjrczr closed this Aug 8, 2024
@Whsjrczr Whsjrczr reopened this Aug 8, 2024
@SigureMo SigureMo changed the title [Typing][PEP585 Upgrade][BUAA][131-150] Use standard collections for type hints for 19 files in test/ [Typing][PEP585 Upgrade][BUAA][131-150] Use standard collections for type hints for 19 files in test/ Aug 8, 2024
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

一个文件都没用 PEP 563?任务描述是怎么看的?没有 PEP 563 怎么升级 PEP 585?

描述里说了先改一个文件,等 approve 后再继续改,然后你直接改了 19 个,是多么自信啊?

所有 PR 都需要重新确认是否使用了 PEP 563,不然 3.8 会挂!

@@ -14,7 +14,7 @@


import math
from typing import Optional, Tuple
from typing import Optional
Copy link
Member

Choose a reason for hiding this comment

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

Optional 保留的理由是?

@@ -70,7 +70,7 @@ def generate_input1(dims, batch, attrs: List[Dict[str, Any]]):

def sample_predictor_configs(
self, program_config
) -> (paddle_infer.Config, List[int], float):
) -> (paddle_infer.Config, list[int], float):
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
) -> (paddle_infer.Config, list[int], float):
) -> tuple[paddle_infer.Config, list[int], float]:

这里是明显错误的,需要修改

别说一个就改一个,其他的也需要改

Copy link
Contributor Author

Choose a reason for hiding this comment

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

抱歉,已全部修改

@paddle-bot paddle-bot bot added the contributor External developers label Aug 8, 2024
@luotao1 luotao1 added the HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 label Aug 9, 2024
SigureMo
SigureMo previously approved these changes Aug 9, 2024
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

LGTMeow 🐾

@SigureMo SigureMo merged commit ac3fb4e into PaddlePaddle:develop Aug 10, 2024
31 checks passed
Jeff114514 pushed a commit to Jeff114514/Paddle that referenced this pull request Aug 14, 2024
…type hints for 19 files in `test/` (PaddlePaddle#67229)

---------

Co-authored-by: SigureMo <sigure.qaq@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants