【Hackathon 6th Fundable Projects 3 No.374】fluid operator tdm_sampler

[co63oc]

PR Category

Others

PR Types

Others

Description

Migrate fluid operator tdm_sampler

[paddle-bot]

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

[wanghuancoder]

LGTM 请 @luotao1 协调日升也Review一下~

[YuanRisheng]

原先Fluid的sampler.h/cc也需要删掉

[co63oc]

原先Fluid的sampler.h/cc也需要删掉

可能其他文件使用，比如paddle/fluid/distributed/index_dataset/index_sampler.h，其他算子迁移后再删除

[YuanRisheng]

原先Fluid的sampler.h/cc也需要删掉

可能其他文件使用，比如paddle/fluid/distributed/index_dataset/index_sampler.h，其他算子迁移后再删除

我看用的地方并不多，可以删掉，然后用到的地方全部改成调用phi下的sampler.h，尽量减少俩套相同代码的存在，减少维护成本

[co63oc]

原先Fluid的sampler.h/cc也需要删掉

可能其他文件使用，比如paddle/fluid/distributed/index_dataset/index_sampler.h，其他算子迁移后再删除

我看用的地方并不多，可以删掉，然后用到的地方全部改成调用phi下的sampler.h，尽量减少俩套相同代码的存在，减少维护成本

不是修改sampler.h就可以，有关联的链接库，phi下的链接库不同

[YuanRisheng]

原先Fluid的sampler.h/cc也需要删掉

可能其他文件使用，比如paddle/fluid/distributed/index_dataset/index_sampler.h，其他算子迁移后再删除

我看用的地方并不多，可以删掉，然后用到的地方全部改成调用phi下的sampler.h，尽量减少俩套相同代码的存在，减少维护成本

不是修改sampler.h就可以，有关联的链接库，phi下的链接库不同

这个应该不复杂吧，链库的话只需要链接一个phi target就可以了，如果说等其他算子迁移删除也行，那就必须优先搞一下， 不然如果拖的时间太长后边有人对sampler修改就容易造成逻辑不一致的问题了，因为后边改的人他未必会俩个地方都改

[co63oc]

原先Fluid的sampler.h/cc也需要删掉

可能其他文件使用，比如paddle/fluid/distributed/index_dataset/index_sampler.h，其他算子迁移后再删除

我看用的地方并不多，可以删掉，然后用到的地方全部改成调用phi下的sampler.h，尽量减少俩套相同代码的存在，减少维护成本

不是修改sampler.h就可以，有关联的链接库，phi下的链接库不同

这个应该不复杂吧，链库的话只需要链接一个phi target就可以了，如果说等其他算子迁移删除也行，那就必须优先搞一下， 不然如果拖的时间太长后边有人对sampler修改就容易造成逻辑不一致的问题了，因为后边改的人他未必会俩个地方都改

好的，那等其他算子迁移修改