Skip to content

Conversation

@OliverLPH
Copy link
Contributor

PR types

Others

PR changes

Others

Describe

update paddle_build.bat for remove op grad

@paddle-bot-old
Copy link

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

set ON_INFER=ON
set WITH_PYTHON=OFF
set CUDA_ARCH_NAME=All
python %work_dir%\tools\remove_grad_op_and_kernel.py
Copy link
Contributor

Choose a reason for hiding this comment

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

下面加一行 if %errorlevel% NEQ 0 exit /b 1 吧,确保这行是正确的执行了,不然脚本有错不会退出

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, thx

增加裁剪op脚本的状态判断
zhwesky2010
zhwesky2010 previously approved these changes Jun 28, 2021
if %errorlevel% NEQ 0 exit /b 1

call :cmake || goto cmake_error
call :build || goto build_error
Copy link
Contributor

Choose a reason for hiding this comment

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

要不在build这行后面加一个 call :test_inference || goto test_inference_error 吧,会做一个demo_ci的预测库检查

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, thx

add inference demo test after build finish
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 1cfa105 into develop Jul 5, 2021
@OliverLPH OliverLPH deleted the OliverLPH-patch-2 branch July 7, 2021 09:02
OliverLPH added a commit to OliverLPH/Paddle that referenced this pull request Jul 21, 2021
…e lib (PaddlePaddle#33800)

1. check status of remove op grad shell scripts
2. Update paddle_build.bat to remove op grad when build windows inference lib
3. add inference demo test after build finish
Superjomn pushed a commit that referenced this pull request Jul 21, 2021
)

* Update paddle_build.bat to remove op grad when build windows inference lib (#33800)

1. check status of remove op grad shell scripts
2. Update paddle_build.bat to remove op grad when build windows inference lib
3. add inference demo test after build finish

* Update paddle_build.sh for remove op grad when compile c++ inference lib (#33798)

* fix zip inference library bug (#34025)

Co-authored-by: Zhou Wei <[email protected]>
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.

3 participants