Skip to content

Conversation

angelayi
Copy link
Contributor

@angelayi angelayi commented Apr 7, 2023

Fixes #68534

Similar to how submodules are added, if there already exists an attribute with the same name in gm as in replacement, the attribute value in gm will take precedence.

@pytorch-bot
Copy link

pytorch-bot bot commented Apr 7, 2023

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/98604

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 Failures

As of commit 02114b6:

NEW FAILURES - The following jobs have failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@pytorch-bot pytorch-bot bot added the release notes: fx release notes category label Apr 7, 2023
@angelayi
Copy link
Contributor Author

angelayi commented Apr 7, 2023

@pytorchbot merge -f "Failed test errors with 'The operation was canceled.' so I don't think it's my fault? sorry in advance if it is my fault"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

skotapati pushed a commit to kulinseth/pytorch that referenced this pull request Apr 10, 2023
Fixes pytorch#68534

Similar to how submodules are added, if there already exists an attribute with the same name in `gm` as in `replacement`, the attribute value in `gm` will take precedence.
Pull Request resolved: pytorch#98604
Approved by: https://github.com/andrewor14, https://github.com/SherlockNoMad
@github-actions github-actions bot deleted the rewrite_bug branch October 7, 2024 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[subgraph_rewriter] Better attribute matching support
4 participants