Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[BE][pytree] rename NodeDef member to match the type annotations: *_fn -> *_func #148474

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 9 commits into
base: gh/XuehaiPan/254/base
Choose a base branch
from

Conversation

XuehaiPan
Copy link
Collaborator

@XuehaiPan XuehaiPan commented Mar 4, 2025

Stack from ghstack (oldest at bottom):

This PR renames the member in NodeDef from *_fn -> *_func. The old names are aliased to the new names and will emit a FutureWarning when accessed.

cc @zou3519

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Mar 4, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit 6bf30c8 with merge base e9e1aac (image):
💚 Looks good so far! There are no failures yet. 💚

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

XuehaiPan added a commit that referenced this pull request Mar 4, 2025
[ghstack-poisoned]
XuehaiPan added a commit that referenced this pull request Mar 4, 2025
@XuehaiPan XuehaiPan changed the title [BE][pytree] rename NodeDef member to match the type annotations [BE][pytree] rename NodeDef member to match the type annotations: *_fn -> *_func Mar 4, 2025
XuehaiPan added a commit that referenced this pull request Mar 4, 2025
…*_fn -> *_func`

ghstack-source-id: d5c6cc4
Pull Request resolved: #148474
@XuehaiPan XuehaiPan requested a review from zou3519 March 4, 2025 20:03
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Mar 4, 2025
…*_fn -> *_func`

ghstack-source-id: d5c6cc4
Pull Request resolved: pytorch#148474
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Mar 4, 2025
…*_fn -> *_func`

ghstack-source-id: d5c6cc4
Pull Request resolved: pytorch#148474
[ghstack-poisoned]
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Mar 12, 2025
…*_fn -> *_func`

ghstack-source-id: e43a4e5
Pull Request resolved: pytorch#148474
[ghstack-poisoned]
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Mar 13, 2025
…*_fn -> *_func`

ghstack-source-id: 4f6f9e1
Pull Request resolved: pytorch#148474
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Mar 13, 2025
…*_fn -> *_func`

ghstack-source-id: 4f6f9e1
Pull Request resolved: pytorch#148474
[ghstack-poisoned]
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Mar 14, 2025
…*_fn -> *_func`

ghstack-source-id: f868b48
Pull Request resolved: pytorch#148474
[ghstack-poisoned]
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Mar 31, 2025
…*_fn -> *_func`

ghstack-source-id: 2cce41e
Pull Request resolved: pytorch#148474
[ghstack-poisoned]
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Apr 1, 2025
…*_fn -> *_func`

ghstack-source-id: bffd00d
Pull Request resolved: pytorch#148474
[ghstack-poisoned]
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Apr 9, 2025
…*_fn -> *_func`

ghstack-source-id: 0bc8be9
Pull Request resolved: pytorch#148474
[ghstack-poisoned]
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request May 2, 2025
…*_fn -> *_func`

ghstack-source-id: 243fdd4
Pull Request resolved: pytorch#148474
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request May 2, 2025
…*_fn -> *_func`

ghstack-source-id: 243fdd4
Pull Request resolved: pytorch#148474
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants