docs icon indicating copy to clipboard operation
docs copied to clipboard

【PaddlePaddle Hackathon 3】16 新增 API paddle.take

Open S-HuaBomb opened this issue 3 years ago • 6 comments

增加 paddle.take 的中文文档。

RFC 的链接为:20220714_api_design_for_take.md 该算子的实现 PR 链接为:https://github.com/PaddlePaddle/Paddle/pull/44741 PADDLEPADDLE_PR=44741

S-HuaBomb avatar Jul 30 '22 04:07 S-HuaBomb

感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-5099.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html 预览工具的更多说明,请参考:[Beta]飞桨文档预览工具

paddle-bot[bot] avatar Jul 30 '22 04:07 paddle-bot[bot]

image

看样子还有些代码风格检查没有过,需要本地安装 pre-commit(这个 repo 里也需要初始化下),之后相应的文件随便加点什么后 commit 以触发 pre-commit 的自动修复(最好是些没什么影响的,比如在某一行后加些空格,这些之后会在触发 pre-commit 后自动删除,如果没有自动删的话手动记得删一下 :joy:)

SigureMo avatar Aug 07 '22 20:08 SigureMo

image

看样子还有些代码风格检查没有过,需要本地安装 pre-commit(这个 repo 里也需要初始化下),之后相应的文件随便加点什么后 commit 以触发 pre-commit 的自动修复(最好是些没什么影响的,比如在某一行后加些空格,这些之后会在触发 pre-commit 后自动删除,如果没有自动删的话手动记得删一下 😂)

很奇怪,我修改 take_cn.rst 之后 commit 却没有触发 pre-commit,环境中已经安装了。不过 take_cn.rst 的问题应该是最后空了两行,我手动删了一行。 THX,done

S-HuaBomb avatar Aug 08 '22 01:08 S-HuaBomb

image 看样子还有些代码风格检查没有过,需要本地安装 pre-commit(这个 repo 里也需要初始化下),之后相应的文件随便加点什么后 commit 以触发 pre-commit 的自动修复(最好是些没什么影响的,比如在某一行后加些空格,这些之后会在触发 pre-commit 后自动删除,如果没有自动删的话手动记得删一下 😂)

很奇怪,我修改 take_cn.rst 之后 commit 却没有触发 pre-commit,环境中已经安装了。不过 take_cn.rst 的问题应该是最后空了两行,我手动删了一行。 THX,done

因为这个格式检查的pr上周五合入的, @SigureMo 贡献的pr。目前看来CI还是会卡住,再检查一下吧~

Ligoml avatar Aug 08 '22 06:08 Ligoml

在新的仓库中使用 pre-commit 需要先运行 pre-commit install

我用 pre-cpmmit 会有这个问题: image 它检查了之前提到的由于大小写出问题的文件,因为它们一直被标为修改文件在暂存区,restore 也没有用。我删除了空了两行的地方,这次应该好了。 Thx,done

S-HuaBomb avatar Aug 08 '22 07:08 S-HuaBomb

在新的仓库中使用 pre-commit 需要先运行 pre-commit install。

嗯嗯,首先全局用 pip install pre-commit 是全局安装这个工具(一次即可)

之后每个 repo 都需要 pre-commit install 将 git hooks 注册到具体的这一个 repo

我用 pre-commit 会有这个问题:

嗯对的,我本地也用不了 pre-commit,现在都是在 codespace 上修改文档开发的(那个 issue 里我也有说),就很麻烦……

SigureMo avatar Aug 08 '22 08:08 SigureMo