Skip to content

Comments

[Dy2St] Skip prune parameter in dy2st and save#68442

Merged
yuanlehome merged 3 commits intoPaddlePaddle:developfrom
cattidea:dy2st/skip-param-prune-in-dy2st-and-save
Sep 27, 2024
Merged

[Dy2St] Skip prune parameter in dy2st and save#68442
yuanlehome merged 3 commits intoPaddlePaddle:developfrom
cattidea:dy2st/skip-param-prune-in-dy2st-and-save

Conversation

@SigureMo
Copy link
Member

@SigureMo SigureMo commented Sep 25, 2024

PR Category

Execute Infrastructure

PR Types

Bug fixes

Description

PIR 下不在动转静+save 阶段 prune 参数和 buffer,只会导致 save 的参数多一些

cc @HydrogenSulfate

Pcard-67164

def __init__(self):
super().__init__()
self.linear = paddle.nn.Linear(7, 10)
self.register_buffer("buffer", paddle.randn([5, 1]))
Copy link
Contributor

Choose a reason for hiding this comment

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

对self.create_parameters也有效吗

Copy link
Member Author

Choose a reason for hiding this comment

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

都有效,结论是 paramter 和 buffer 两者都不剪枝

Copy link
Contributor

Choose a reason for hiding this comment

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

都有效,结论是 paramter 和 buffer 两者都不剪枝

好的

Copy link
Contributor

@yuanlehome yuanlehome left a comment

Choose a reason for hiding this comment

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

LGTM

@yuanlehome yuanlehome merged commit d5fd91c into PaddlePaddle:develop Sep 27, 2024
@SigureMo SigureMo deleted the dy2st/skip-param-prune-in-dy2st-and-save branch September 27, 2024 09:50
xiaoguoguo626807 added a commit that referenced this pull request Apr 15, 2025
* fix pir program prune

* fix pir program prune

* remoce print

* no param load
YqGe585 pushed a commit to YqGe585/Paddle that referenced this pull request May 7, 2025
…ePaddle#72202)

* fix pir program prune

* fix pir program prune

* remoce print

* no param load
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