Skip to content

Conversation

@SigureMo
Copy link
Member

@SigureMo SigureMo commented May 14, 2024

PR Category

Execute Infrastructure

PR Types

Not User Facing

Description

#64220 添加了默认不拆 matmul_grad 的策略以保证性能,但科学计算场景是需要拆 matmul_grad 以确保高阶微分的正确性的,因此在 test_paddlescience 里添加 FLAGS_prim_vjp_skip_default_ops=False 以强制开启 matmul_grad 的拆解

本 PR 应可修复 PR-CI-Py3-PIR 流水线挂掉的 test_paddlescience

PCard-66972

@paddle-bot
Copy link

paddle-bot bot commented May 14, 2024

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

@SigureMo SigureMo requested a review from gouzil May 14, 2024 08:28
gouzil
gouzil previously approved these changes May 14, 2024
@SigureMo SigureMo requested a review from gouzil May 14, 2024 13:47
gouzil
gouzil previously approved these changes May 14, 2024
@Aurelius84 Aurelius84 merged commit 396ffc6 into PaddlePaddle:develop May 15, 2024
@SigureMo SigureMo deleted the pir/force-decomp-matmul-grad-in-test-paddlescience branch May 15, 2024 01:58
co63oc pushed a commit to co63oc/Paddle that referenced this pull request May 15, 2024
…lt_ops=False` in `test_paddlescience` (PaddlePaddle#64286)

* [PIR] Force decomp `matmul_grad` OP by use `FLAGS_prim_vjp_skip_default_ops=False` in `test_paddlescience`

* use os.environ

* set env before import paddle

* add env to cmake
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