Skip to content

[Bug Fix] Use ast to generate the hash of Paddle models.#266

Merged
JewelRoam merged 2 commits into
PaddlePaddle:developfrom
Xreki:fix_paddle_hash
Sep 9, 2025
Merged

[Bug Fix] Use ast to generate the hash of Paddle models.#266
JewelRoam merged 2 commits into
PaddlePaddle:developfrom
Xreki:fix_paddle_hash

Conversation

@Xreki

@Xreki Xreki commented Sep 8, 2025

Copy link
Copy Markdown
Collaborator

PR Category

Bug Fix

Description

  1. 修正Paddle模型样本hash值的生成方式,改成:
  • 使用ast获取Paddle样本模型的forward函数源码,来生成Paddle模型的hash值。
  • 并通过ast.unparse(fn)会过滤掉原样本文件中的注释。
  1. 修改validate中哈希值校验的逻辑,修改成:
  • 如果已经存在graph_hash.txt文件,则判断新生成的hash值跟graph_hash.txt文件中的哈希值是否一致。若不一致,先更新graph_hash.txt文件并报错。
  • 如果不存在graph_hash.txt文件,且args.dump_graph_hash_keyFalse,报错。
  • args.dump_graph_hash_key的默认值改成True,确保CI上对hash值的正确性、一致性进行检查。
  1. 更新所有Paddle样本的hash值。注意:当前存在一些hash相同的重复样本,本PR只更新hash值,下一个PR再删除冗余样本。

@paddle-bot

paddle-bot Bot commented Sep 8, 2025

Copy link
Copy Markdown

Thanks for your contribution!

@Xreki Xreki requested a review from lixinqi September 9, 2025 02:52
@JewelRoam JewelRoam merged commit 24567e3 into PaddlePaddle:develop Sep 9, 2025
3 checks passed
@Xreki Xreki deleted the fix_paddle_hash branch September 9, 2025 03:26
JewelRoam pushed a commit to JewelRoam/GraphNet that referenced this pull request Oct 29, 2025
…e#266)

* Use ast to generate the hash of Paddle models.

* Update all the hash value of paddle samples.
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