-
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
【Hackathon 7th No.28】为 paddle.clip 进行功能增强 #68819
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
|
你好,我想请问下,对于C++的DenseTensor算子,我该去哪里查看他的实现,比如矩阵数据的表现形式,以及GPU算子注册的实现,我看了那些cu文件,并没有DenseTensor,我是打算将输入全都换成Tensor,然后再重新clip,或者是否函数重载的形式?我并没有找到对应的案例。 |
那你需要看的内容还比较多,需要熟悉相关源码,这个是开发流程和部分代码解读 |
我目前的思路是这样的:将x、min、max全部转为tensor之后,在C++层将他们全都转为一维的矩阵形式,然后逐个遍历之后再转回来。 |
|
请问如果按照我这个思路,我其实是不需要在cpu、gpu那里增加代码,而是只需要修改impl的代码的对吧? |
我发现改底层的话需要修改的太多了,而且有些我还不了解,我就是用算子组合的形式了。使用expand+where进行实现,效果是一样的 |
请问静态图如何判断传入的参数是否为Tensor? @zhwesky2010 |
@luotao1 @zhwesky2010 请问能否帮忙看一下,我觉得我新增的test他并没有执行,因为我本地是可以的,然后还有就是这个 |
是docker镜像挂了? @luotao1 |
已经都好了,请检查一下 @zhwesky2010 |
请问为什么我使用staric.data创建的静态图tensor不算是variable类型吗?而且我添加的代码是都已经跑过了的,他还是说覆盖率不够 @zhwesky2010 |
#69193 |
这个还是不能在python层写,需改造已有的kernel |
PR Category
Inference
PR Types
Others
Description
为 paddle.clip 进行功能增强
PaddlePaddle/docs#6924