Skip to content

fix(patch): remove duplicate @patch_function on _patch_graph_context_manager#62

Open
popsiclexu wants to merge 3 commits intoMooreThreads:mainfrom
popsiclexu:xzx/fix-dup-patch
Open

fix(patch): remove duplicate @patch_function on _patch_graph_context_manager#62
popsiclexu wants to merge 3 commits intoMooreThreads:mainfrom
popsiclexu:xzx/fix-dup-patch

Conversation

@popsiclexu
Copy link
Copy Markdown
Contributor

@popsiclexu popsiclexu commented May 8, 2026

Summary

  • Remove @patch_function decorator from _patch_graph_context_manager to prevent it from being patched twice

@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented May 8, 2026

🤖 Augment PR Summary

Summary: Removes a redundant @patch_function registration on _patch_graph_context_manager so it isn’t invoked twice during patch application.

Changes:

  • Drops the decorator from _patch_graph_context_manager, keeping it callable from _patch_torch_cuda_module without being separately registered

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@yeahdongcn
Copy link
Copy Markdown
Collaborator

@popsiclexu Could you rerun all unit tests to make sure there is no regression? Thanks!

snow_xu and others added 2 commits May 8, 2026 17:06
Signed-off-by: Xiaodong Ye <xiaodong.ye@mthreads.com>
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