[Bug Fix] Fix SymInt issue in subgraph decomposition #506
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks for your contribution! |
Collaborator
|
请提交这个pr前后的ES指标对比。 |
Contributor
Author
测试脚本 https://github.com/PaddlePaddle/GraphNet/blob/develop/graph_net/test/fx_graph_module_unserialize_test.sh |
lixinqi
approved these changes
Dec 29, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Category
Bug Fix
Description
修复了子图分解部分case失败的问题
在Dynamo捕捉计算图的过程中，s0，s1等symint（0-d tensor，torch.int64）因为没有被实际使用而会被优化掉，最后捕捉到的计算图会去掉这些输入，但是我们创建的input中是包含着的，需要补全回来
之前的逻辑是只补全一个symint参数，而现在是将所有的symint全量补全，修复了symint大于1的例子（如googlenet中存在两个symint参数）
本PR中，子图分解部分会大幅度提高成功率