Skip to content

support infer_ut on windows nightly build#35049

Merged
OliverLPH merged 3 commits intoPaddlePaddle:developfrom
OliverLPH:enable_windows_infer_ut
Aug 23, 2021
Merged

support infer_ut on windows nightly build#35049
OliverLPH merged 3 commits intoPaddlePaddle:developfrom
OliverLPH:enable_windows_infer_ut

Conversation

@OliverLPH
Copy link
Contributor

PR types

Others

PR changes

Others

Describe

  1. 修改infer_ut 执行脚本,兼容Windows平台
  2. paddle_build.sh build_inference_lib 标签下增加启动代码

@OliverLPH OliverLPH requested a review from zhwesky2010 August 20, 2021 11:42
@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@OliverLPH OliverLPH requested a review from jiweibo August 20, 2021 11:42
Copy link
Contributor

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

本地测了,能跑过吗?后续如果这个标签没问题,看看时间,能否合并到test_inference标签里给CI跑

@OliverLPH
Copy link
Contributor Author

本地测了,能跑过吗?后续如果这个标签没问题,看看时间,能否合并到test_inference标签里给CI跑

本地是通过 busybox 手动执行 run.sh 能够跑过,没有带paddle_build.bat里面编译的流程,所以想试下上流水线能不能跑过

@zhwesky2010 zhwesky2010 changed the title enable infer_ut on windows nightly build support infer_ut on windows nightly build Aug 23, 2021
else
wget -q --no-proxy ${url_prefix}/${model_name}.tgz
tar xzf *.tgz
if [ $WIN_DETECT != "" ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

Linux用代理能下载过吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

linux用代理可以下载

Copy link
Contributor

Choose a reason for hiding this comment

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

这段逻辑是windows用代理,Linux不用代理

Copy link
Contributor Author

Choose a reason for hiding this comment

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

目前修改为
Linux下:wget -q --no-proxy 取消代理
Windows下:使用busybox,wget -q -Y off 取消代理

-DWITH_GTEST=ON \
-DCMAKE_CXX_FLAGS='/std:c++17' \
-DCMAKE_BUILD_TYPE=Release
msbuild /maxcpucount /property:Configuration=Release ALL_BUILD.vcxproj;
Copy link
Contributor

Choose a reason for hiding this comment

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

这个行尾有分号吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

是多写了分号,但目前不影响 busybox bash run.sh 运行

Copy link
Contributor

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

LGTM

@OliverLPH OliverLPH merged commit 4f86aae into PaddlePaddle:develop Aug 23, 2021
@OliverLPH OliverLPH deleted the enable_windows_infer_ut branch August 23, 2021 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants