Skip to content

[SWE-Paddle] Add task PaddlePaddle__Paddle_74491 - #1456

Open
Echo-Nie wants to merge 5 commits into
PaddlePaddle:masterfrom
Echo-Nie:74491
Open

[SWE-Paddle] Add task PaddlePaddle__Paddle_74491#1456
Echo-Nie wants to merge 5 commits into
PaddlePaddle:masterfrom
Echo-Nie:74491

Conversation

@Echo-Nie

Copy link
Copy Markdown
Contributor

关联

说明

  • 新增 SWE-Paddle 任务 PaddlePaddle__Paddle-74491

@sunzhongkai588

验证结果

状态 Existing dim P2P requires_grad state F2P Invalid assignment F2P
Base + tests/test.patch PASS FAIL,符合预期 FAIL,符合预期
Base + test patch + solution/code.patch PASS PASS PASS

Base:

P2P: 1 passed
F2P: 2 failed, 1 passed
tests/test.sh exit code: 1

Solution:

P2P: PASS
F2P: PASS
Full target test: 3 passed in 0.08s
tests/test.sh: 3 passed in 0.08s


Paddle 目前通过 `stop_gradient` 控制 Tensor 是否参与梯度计算,但还不支持 `requires_grad` 属性。

需要为动态图 Tensor、静态图 Variable 和 PIR Value 增加 `requires_grad`。该属性与 `stop_gradient` 的含义相反:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PIR Value 是啥?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

一个类型?相当于就是PIR 的一个Tensor-like 的符号对象

image


## 详细描述

Paddle 目前通过 `stop_gradient` 控制 Tensor 是否参与梯度计算,但还不支持 `requires_grad` 属性。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

但还不支持 requires_grad 属性。

感觉这句话有点怪,要不加一句为了增加兼容性之类的?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

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