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

转换规则 No. 253/261/279 #223

Merged
merged 1 commit into from
Aug 21, 2023
Merged

Conversation

co63oc
Copy link
Contributor

@co63oc co63oc commented Aug 6, 2023

PR Docs

#112

映射文档 PaddlePaddle/docs#6047

测试框架不支持torch.autograd.profiler.profile转换,修改为unsupport

253 torch.autograd.profiler.profile.export_chrome_trace
261 torch.special.i0e
279 torch.nn.functional.gaussian_nll_loss

PR APIs

253 torch.autograd.profiler.profile.export_chrome_trace
261 torch.special.i0e
279 torch.nn.functional.gaussian_nll_loss

@paddle-bot
Copy link

paddle-bot bot commented Aug 6, 2023

Thanks for your contribution!

@paddle-bot paddle-bot bot added contributor External developers status: proposed labels Aug 6, 2023
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Aug 7, 2023
@luotao1 luotao1 assigned dzhwinter, luotao1 and zhwesky2010 and unassigned dzhwinter Aug 7, 2023
@@ -3087,6 +3087,16 @@
"only_inputs"
]
},
"torch.autograd.profiler.profile.export_chrome_trace": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个没有加单测吗?

Copy link
Contributor Author

@co63oc co63oc Aug 16, 2023

Choose a reason for hiding this comment

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

文档规则可以转换,torch.autograd.profiler.profile 前缀路径测试框架不能识别转换
测试用例
tests/test_autograd_profiler_profile_export_chrome_trace.py

Copy link
Collaborator

Choose a reason for hiding this comment

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

文档规则可以转换,torch.autograd.profiler.profile 前缀路径测试框架不能识别转换 测试用例 tests/test_autograd_profiler_profile_export_chrome_trace.py

好的,这个是一个识别问题,我来解决吧,单测先这样放上去就可以

@zhwesky2010
Copy link
Collaborator

zhwesky2010 commented Aug 18, 2023

@co63oc 需要处理一下冲突,就能合入了

@co63oc
Copy link
Contributor Author

co63oc commented Aug 18, 2023

@co63oc 需要处理一下冲突,就能合入了

已修改冲突

Copy link
Collaborator

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

LGTM

@zhwesky2010 zhwesky2010 merged commit d649fb9 into PaddlePaddle:master Aug 21, 2023
@co63oc co63oc deleted the api253 branch September 7, 2023 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource 快乐开源活动issue与PR status: proposed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants