[PIR] Fix `check_view_api_used_by_inplace` - Githubissues

PaddlePaddle / Paddle

PArallel Distributed Deep LEarning: Machine Learning Framework from Industrial Practice （『飞桨』核心框架，深度学习&机器学习高性能单机、分布式训练和跨平台部署）

http://www.paddlepaddle.org/

Apache License 2.0

22.28k stars 5.61k forks source link

[PIR] Fix `check_view_api_used_by_inplace` #69615

Closed ooooo-create closed 8 hours ago

ooooo-create commented 17 hours ago

PR Category

Execute Infrastructure

PR Types

Others

Description

https://github.com/PaddlePaddle/PaddleDetection/blob/0914611178e9db34e746a351c7fc090048eeebc0/ppdet/modeling/losses/ssd_loss.py#L115

修复检查逻辑， value.detach 使用的 pd_op.share_data_ 不会和其他 inplace api 一样立刻修改显存
但是产生的新的 Value 在逻辑上是不保证安全的，因为之后可能会被其他 inplace api 使用，也修改了之前的显存，目前的检查没有包含这种情况（现在只是跳过了

z = paddle.squeeze(x,-1)
y = x.detach() #不会立刻导致 z 的显存变化
# y.add_(xxx) ，就会导致 z 的显存变化

paddle-bot[bot] commented 17 hours ago

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