Skip to content

[Bug Fix] new sample STFPM to validate fix_hash#273

Merged
Xreki merged 2 commits into
PaddlePaddle:developfrom
HeyDavid633:hash_fix
Sep 12, 2025
Merged

[Bug Fix] new sample STFPM to validate fix_hash#273
Xreki merged 2 commits into
PaddlePaddle:developfrom
HeyDavid633:hash_fix

Conversation

@HeyDavid633

Copy link
Copy Markdown
Contributor

PR Category

Bug Fix
修正了由 python 版本带来的 hash 区别

Description

同时提交新模型 STFPM ,原本的模型dump下来缺少 shadow_output。通过修改 model.py 然后经过 GraphNet validate 达到了入库条件,修改备注如下:

  1. exec_programs.py 中有 6 个 Pirprogram, 但是有 2 个带 grad 已排除,另外 4 个结构完全相同 --- 所以模型只生成了一个 model,py 而不是 subgraph_X * 4个
  2. exec_programs.py 在(之前)出错的点,就是没有 shadow_output;如果修改就是 4 处 Pirprogram 都改,所以选择修改model.py
  3. 生成该模型的 graph_hash.txt 应用了最近版本的 hash 方法;基于PR [Bug Fix] Use astor instead of ast to unparse source code of paddle model, to avoid unstable result between different python versions. #272

@paddle-bot

paddle-bot Bot commented Sep 11, 2025

Copy link
Copy Markdown

Thanks for your contribution!

@HeyDavid633 HeyDavid633 changed the title Hash fix [Bug Fix] new sample STFPM to validate fix_hash Sep 11, 2025
@Xreki Xreki merged commit c1bc381 into PaddlePaddle:develop Sep 12, 2025
3 checks passed
JewelRoam pushed a commit to JewelRoam/GraphNet that referenced this pull request Oct 29, 2025
* Use astor instead of ast to unparse source code of paddle model, to avoid unstable result between different python versions.

* fix hash bug cased by python-version 3.10/3.12
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