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

[CodeStyle] upgrade Lucas-C/pre-commit-hooks in .pre-commit-config.yaml #6240

Merged
merged 2 commits into from
Oct 22, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ body:
attributes:
label: 1.导入分布式训练需要的依赖包
description: |
针对 导入分布式训练需要的依赖 这一环节,请回答以下内容
针对 导入分布式训练需要的依赖 这一环节,请回答以下内容
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
针对 导入分布式训练需要的依赖 这一环节,请回答以下内容
针对导入分布式训练需要的依赖这一环节,请回答以下内容

改成这样吧,下同

value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -74,7 +74,7 @@ body:
attributes:
label: 2.初始化分布式训练环境
description: |
针对 初始化分布式训练环境 这一环节,请回答以下内容
针对 初始化分布式训练环境 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -88,7 +88,7 @@ body:
attributes:
label: 3.设置分布式训练需要的优化器
description: |
针对 设置分布式训练需要的优化器 这一环节,请回答以下内容
针对 设置分布式训练需要的优化器 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -115,7 +115,7 @@ body:
- type: textarea
id: distributed_dp_eval5
attributes:
label: 5.构建训练代码
label: 5.构建训练代码
description: |
针对 构建训练代码 这一环节,请回答以下内容
value: |
Expand Down Expand Up @@ -197,7 +197,7 @@ body:
attributes:
label: 1.导入分布式训练需要的依赖包
description: |
针对 导入分布式训练需要的依赖 这一环节,请回答以下内容
针对 导入分布式训练需要的依赖 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -211,7 +211,7 @@ body:
attributes:
label: 2.初始化分布式训练环境
description: |
针对 初始化分布式训练环境 这一环节,请回答以下内容
针对 初始化分布式训练环境 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -225,7 +225,7 @@ body:
attributes:
label: 3.设置分布式训练需要的优化器
description: |
针对 设置分布式训练需要的优化器 这一环节,请回答以下内容
针对 设置分布式训练需要的优化器 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -252,7 +252,7 @@ body:
- type: textarea
id: distributed_dp_eval15
attributes:
label: 5.构建训练代码
label: 5.构建训练代码
description: |
针对 构建训练代码 这一环节,请回答以下内容
value: |
Expand Down
12 changes: 6 additions & 6 deletions .github/ISSUE_TEMPLATE/4_eval_distributed_PS_docs-issue.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ body:
attributes:
label: 1.导入分布式训练需要的依赖包
description: |
针对 导入分布式训练需要的依赖 这一环节,请回答以下内容
针对 导入分布式训练需要的依赖 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -63,7 +63,7 @@ body:
attributes:
label: 2.定义分布式模式并初始化分布式训练环境
description: |
针对 定义分布式模式并初始化分布式训练环境 这一环节,请回答以下内容
针对 定义分布式模式并初始化分布式训练环境 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -77,7 +77,7 @@ body:
attributes:
label: 3.加载模型
description: |
针对 加载模型 这一环节,请回答以下内容
针对 加载模型 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand Down Expand Up @@ -173,7 +173,7 @@ body:
attributes:
label: 1.导入分布式训练需要的依赖包
description: |
针对 导入分布式训练需要的依赖 这一环节,请回答以下内容
针对 导入分布式训练需要的依赖 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -187,7 +187,7 @@ body:
attributes:
label: 2.定义分布式模式并初始化分布式训练环境
description: |
针对 定义分布式模式并初始化分布式训练环境 这一环节,请回答以下内容
针对 定义分布式模式并初始化分布式训练环境 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand All @@ -201,7 +201,7 @@ body:
attributes:
label: 3.加载模型
description: |
针对 加载模型 这一环节,请回答以下内容
针对 加载模型 这一环节,请回答以下内容
value: |
- 完成情况(成功/不成功) :
- 遇到问题:
Expand Down
10 changes: 5 additions & 5 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,16 +16,16 @@ repos:
- id: trailing-whitespace
files: \.md$|\.rst$
- repo: https://github.com/Lucas-C/pre-commit-hooks
rev: v1.1.14
rev: v1.5.1
hooks:
- id: forbid-crlf
files: \.md$|\.rst$
types: [text]
- id: remove-crlf
files: \.md$|\.rst$
types: [text]
- id: forbid-tabs
files: \.md$|\.rst$
types: [text]
- id: remove-tabs
files: \.md$|\.rst$
types: [text]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

两个 forbid 可以删掉,因为和 remove 重复了,可以参考 Paddle 那边的,同样,下面的 check-whitespace-between-cn-and-en-char 之后也可以删掉,因为 insert-whitespace-between-cn-and-en-char 会既检查又修复

- repo: https://github.com/ShigureLab/dochooks
rev: v0.3.0
hooks:
Expand Down
12 changes: 6 additions & 6 deletions ci_scripts/gendoc.sh
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,10 @@ if [ -f ${FLUIDDOCDIR}/ci_scripts/hooks/pre-doc-compile.sh ] ; then
fi

thread=2
tmp_fifofile=/tmp/$$.fifo #脚本运行的当前进程ID号作为文件名
mkfifo $tmp_fifofile #新建一个随机fifo管道文件
exec 6<>$tmp_fifofile #定义文件描述符6指向这个fifo管道文件
rm $tmp_fifofile #清空管道内容
tmp_fifofile=/tmp/$$.fifo #脚本运行的当前进程ID号作为文件名
mkfifo $tmp_fifofile #新建一个随机fifo管道文件
exec 6<>$tmp_fifofile #定义文件描述符6指向这个fifo管道文件
rm $tmp_fifofile #清空管道内容
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
tmp_fifofile=/tmp/$$.fifo #脚本运行的当前进程ID号作为文件名
mkfifo $tmp_fifofile #新建一个随机fifo管道文件
exec 6<>$tmp_fifofile #定义文件描述符6指向这个fifo管道文件
rm $tmp_fifofile #清空管道内容
tmp_fifofile=/tmp/$$.fifo # 脚本运行的当前进程ID号作为文件名
mkfifo $tmp_fifofile # 新建一个随机fifo管道文件
exec 6<>$tmp_fifofile # 定义文件描述符6指向这个fifo管道文件
rm $tmp_fifofile # 清空管道内容


# for循环 往 fifo管道文件中写入$thread个空行
for ((i=0;i<$thread;i++));do
Expand Down Expand Up @@ -68,8 +68,8 @@ for lang in en zh ; do
} &
done

wait #等到后台的进程都执行完毕
exec 6>&- ##删除文件描述符6
wait #等到后台的进程都执行完毕
exec 6>&- ##删除文件描述符6
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
wait #等到后台的进程都执行完毕
exec 6>&- ##删除文件描述符6
wait # 等到后台的进程都执行完毕
exec 6>&- # 删除文件描述符6


if [ -f ${FLUIDDOCDIR}/ci_scripts/hooks/post-doc-compile.sh ] ; then
${FLUIDDOCDIR}/ci_scripts/hooks/post-doc-compile.sh ${OUTPUTDIR} ${VERSIONSTR}
Expand Down
16 changes: 8 additions & 8 deletions docs/design/mkldnn/acquire_api/scripts/acquire.dot
Original file line number Diff line number Diff line change
Expand Up @@ -8,27 +8,27 @@ nodesep=1

node[width=4.4,shape=box]

Node0x490c380 [shape=record,label="SoftmaxMKLDNNKernel::Compute()\l"];
Node0x490c380 [shape=record,label="SoftmaxMKLDNNKernel::Compute()\l"];

Node0x4ab38f0 [shape=record,label="MKLDNNActivationKernel::Compute()\l"];
Node0x4ab38f0 [shape=record,label="MKLDNNActivationKernel::Compute()\l"];

subgraph cluster_A {
label="Derived Handlers"
node[width=7.4,shape=box]
style=dotted
// Dummy[shape=record,label="", color=invis];
Node0x4915e90 [shape=record,label="SoftmaxMKLDNNHandler::SoftmaxMKLDNNHandler\<forward\>()\l"];
Node0x4b2e4f0 [shape=record,label="ActivationMKLDNNHandler::ActivationMKLDNNHandler\<forward\>()\l"];
Node0x4915e90 [shape=record,label="SoftmaxMKLDNNHandler::SoftmaxMKLDNNHandler\<forward\>()\l"];
Node0x4b2e4f0 [shape=record,label="ActivationMKLDNNHandler::ActivationMKLDNNHandler\<forward\>()\l"];
}

subgraph cluster_B {
label="Base MKLDNNHandler"
style=dotted
node[width=6.2,shape=box]
Node0x49164c0 [shape=record,label="MKLDNNHandlerT::AcquireSrcMemory()\l"];
Dst[shape=record,label="MKLDNNHandlerT::AcquireDstMemory()\l"];
Node0x491bca0 [shape=record,label="MKLDNNHandlerT::AcquireForwardPrimitive()\l"];
Node0x496cfc0 [shape=record,label="MKLDNNHandlerT::AcquireForwardPrimitiveDescriptor()\l"];
Node0x49164c0 [shape=record,label="MKLDNNHandlerT::AcquireSrcMemory()\l"];
Dst[shape=record,label="MKLDNNHandlerT::AcquireDstMemory()\l"];
Node0x491bca0 [shape=record,label="MKLDNNHandlerT::AcquireForwardPrimitive()\l"];
Node0x496cfc0 [shape=record,label="MKLDNNHandlerT::AcquireForwardPrimitiveDescriptor()\l"];
}

Node0x490c380 -> Node0x4915e90[style="bold"];
Expand Down
Loading