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

[CustomOp] Fix PlaceType related compat error #41826

Merged

Conversation

chenwhql
Copy link
Contributor

@chenwhql chenwhql commented Apr 14, 2022

PR types

Bug fixes

PR changes

APIs

Describe

Fix PlaceType related compat error

PlaceType枚举类型的移除导致多卡模型中自定义算子执行均会失败,为避免不兼容升级,重新恢复这个枚举类型,后续通过发起不兼容升级评审删除

@paddle-bot-old
Copy link

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

@chenwhql chenwhql changed the title Fix PlaceType related compat error [CustomOp] Fix PlaceType related compat error Apr 14, 2022
@chenwhql chenwhql merged commit b5d9c31 into PaddlePaddle:develop Apr 17, 2022
chenwhql added a commit to chenwhql/Paddle that referenced this pull request Apr 17, 2022
* fix place type related compat error

* fix test failed

* remove dll decl

* revert place type change

* add dll decl
phlrain pushed a commit that referenced this pull request Apr 18, 2022
* [Phi&CustomOp] Remove deprecated enum PlaceType for custom op & add warning (#41647)

* remove old custom op placetype

* replace dist  placetype using

* add with gpu macro

* fix mutable_data error

* fix set value error

* add comment

* remove all is initialized using (#41766)

* remove inner_place using (#41768)

* polish tensor depreacted method warning (#41807)

* [CustomOp] Fix PlaceType related compat error (#41826)

* fix place type related compat error

* fix test failed

* remove dll decl

* revert place type change

* add dll decl

* resolve conflict
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants