首页 Paddle框架 帖子详情
transpose_mkldnn Op中调用了ShareDataWith方法
收藏
快速回复
Paddle框架 问答深度学习炼丹技巧 681 9
transpose_mkldnn Op中调用了ShareDataWith方法
收藏
快速回复
Paddle框架 问答深度学习炼丹技巧 681 9

Paddle框架的OP开发规范中规定Op禁止调用ShareDataWith和ShareBufferWith方法,但在transpose_mkldnn OP的文件transpose_mkldnn_op.cc中仍存在对输入和输出调用ShareDateWith方法。想请教一下,这个是否需要根据OP开发规范将其修改为TensorCopy的形式,还是出于什么考量仍需保留ShareDateWith方法呢?

image
image

0
收藏
回复
全部评论(9)
时间顺序
AIStudio791310
#2 回复于2020-02

这是Op规范的存量问题吗?
发这个issue的目的是,需要商讨?还是发给负责人修改的?

0
回复
AIStudio786549
#3 回复于2020-02

嗯呢,是Op规范的存量问题。
因我暂未找到这个Op的负责人,所以提这个issue想商讨一下该Op是否可以按照Op禁止调用ShareDataWith和ShareBufferWith方法中的建议进行修改?

0
回复
AIStudio791310
#4 回复于2020-02

好的,了解了

0
回复
AIStudio785465
#5 回复于2020-02
@bingyanghuang

请 帮忙看和翻译一下

0
回复
AIStudio786090
#6 回复于2020-02
@gfwm2013

Can you please take a look at conv_mkldnn_op.cc? Because there is also ShareDataWith in that op. Should we also remove ShareDataWith in conv_mkldnn_op.cc ?

0
回复
AIStudio786549
#7 回复于2020-02
@lidanqing-intel

Please remove ShareDataWith in conv_mkldnn_op.cc.

0
回复
AIStudio786090
#8 回复于2020-02
@lidanqing-intel

Please remove ShareDataWith in conv_mkldnn_op.cc.

Ok

0
回复
AIStudio786090
#9 回复于2020-02
@lidanqing-intel

Please remove ShareDataWith in conv_mkldnn_op.cc.

Ok

0
回复
AIStudio786090
#10 回复于2020-02

PR #22591 merged

0
回复
需求/bug反馈?一键提issue告诉我们
发现bug?如果您知道修复办法,欢迎提pr直接参与建设飞桨~
在@后输入用户全名并按空格结束,可艾特全站任一用户