Skip to content
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

add remove_duplicate flag to named_parameters #759

Closed
wants to merge 1 commit into from

Conversation

samdow
Copy link
Contributor

@samdow samdow commented Oct 28, 2022

Summary: Since the remove_duplicate flag was added to named_buffers in D39493161 (672d105), this adds the same flag to named_parameters

Differential Revision: D40801899

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Oct 28, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40801899

samdow added a commit to samdow/torchrec that referenced this pull request Oct 31, 2022
Summary:
Pull Request resolved: pytorch#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@672d105), this adds the same flag to named_parameters

Differential Revision: D40801899

fbshipit-source-id: a01807c81f0069ea02bdb95d3f937d458f3d9a5d
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40801899

samdow added a commit to samdow/pytorch that referenced this pull request Oct 31, 2022
Summary:
X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Differential Revision: D40801899

fbshipit-source-id: d3054bd9659d6f63f22a1d55fb0dc8898674ad87
samdow added a commit to samdow/torchrec that referenced this pull request Nov 1, 2022
Summary:
X-link: pytorch/pytorch#88090

Pull Request resolved: pytorch#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@672d105), this adds the same flag to named_parameters

Differential Revision: D40801899

fbshipit-source-id: 083fc56e1b8064faf59e2ce4695741dfc150a5d5
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40801899

samdow added a commit to samdow/pytorch that referenced this pull request Nov 1, 2022
Summary:
Pull Request resolved: pytorch#88090

X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Differential Revision: D40801899

fbshipit-source-id: 1b5eb2c4a4b51414256518f648faaff610f78e12
samdow added a commit to samdow/pytorch that referenced this pull request Nov 1, 2022
Summary:
Pull Request resolved: pytorch#88090

X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Differential Revision: D40801899

fbshipit-source-id: ece48c40c17de93b5e0654ede4574870927fc2f1
samdow added a commit to samdow/torchrec that referenced this pull request Nov 1, 2022
Summary:
X-link: pytorch/pytorch#88090

Pull Request resolved: pytorch#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@672d105), this adds the same flag to named_parameters

Differential Revision: D40801899

fbshipit-source-id: 8676b5e5074924708f85d153cff7d20096a8c7a2
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40801899

samdow added a commit to samdow/pytorch that referenced this pull request Nov 1, 2022
Summary:
Pull Request resolved: pytorch#88090

X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Differential Revision: D40801899

fbshipit-source-id: 51e8e0b00f8e6ca52f08d0752ab5b5838b74cd50
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40801899

samdow added a commit to samdow/torchrec that referenced this pull request Nov 7, 2022
Summary:
X-link: pytorch/pytorch#88090

Pull Request resolved: pytorch#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@672d105), this adds the same flag to named_parameters

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 4b537c942650cbcbdfac57ba54040bbd2c0e29f7
samdow added a commit to samdow/pytorch that referenced this pull request Nov 7, 2022
Summary:
Pull Request resolved: pytorch#88090

X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 872b28f2f0bfc667ec9065353d4db45c272a8064
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40801899

samdow added a commit to samdow/torchrec that referenced this pull request Nov 7, 2022
Summary:
X-link: pytorch/pytorch#88090

Pull Request resolved: pytorch#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@672d105), this adds the same flag to named_parameters

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 726ea2fd62dda164d0bd7616a85ce7eb1d473507
samdow added a commit to samdow/pytorch that referenced this pull request Nov 7, 2022
Summary:
Pull Request resolved: pytorch#88090

X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 64b42977ccaf231c03b524d54826e35104c630c9
samdow added a commit to samdow/torchrec that referenced this pull request Nov 7, 2022
Summary:
X-link: pytorch/pytorch#88090

Pull Request resolved: pytorch#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@672d105), this adds the same flag to named_parameters

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 3b195a4c00c38636ccddf7313c0dc3ea976f0ad5
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40801899

samdow added a commit to samdow/pytorch that referenced this pull request Nov 7, 2022
Summary:
Pull Request resolved: pytorch#88090

X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 2553cceecae1f37c544275d9841dff0097494813
samdow added a commit to samdow/torchrec that referenced this pull request Nov 7, 2022
Summary:
X-link: pytorch/pytorch#88090

Pull Request resolved: pytorch#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@672d105), this adds the same flag to named_parameters

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 2adc53cebb5658c639b429bab24d69a106959ff6
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40801899

samdow added a commit to samdow/pytorch that referenced this pull request Nov 7, 2022
Summary:
Pull Request resolved: pytorch#88090

X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 99aad6bb4b4876a81a6a92785f3e40a1c006538a
Summary:
X-link: pytorch/pytorch#88090

Pull Request resolved: pytorch#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@672d105), this adds the same flag to named_parameters

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 89ab659ebdc2f916e62609f6e793d426b3a4518b
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40801899

samdow added a commit to samdow/pytorch that referenced this pull request Nov 8, 2022
Summary:
Pull Request resolved: pytorch#88090

X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 163bfd5052c42b9bd87cb35931d9f7be627e3256
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Nov 9, 2022
Summary:
X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Differential Revision: D40801899

Pull Request resolved: #88090
Approved by: https://github.com/albanD
lequytra pushed a commit to lequytra/torchrec that referenced this pull request Dec 6, 2022
Summary:
X-link: pytorch/pytorch#88090

Pull Request resolved: pytorch#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@672d105), this adds the same flag to named_parameters

Reviewed By: albanD

Differential Revision: D40801899

fbshipit-source-id: 5318259adc6e3c487ccd95aeefecf0a02a1ce79d
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
…orch#88090)

Summary:
X-link: pytorch/torchrec#759

Since the remove_duplicate flag was added to named_buffers in D39493161 (pytorch@c12f829), this adds the same flag to named_parameters

Test Plan:
python test/test_nn.py -k test_buffers_and_named_buffers

OSS Tests

Differential Revision: D40801899

Pull Request resolved: pytorch#88090
Approved by: https://github.com/albanD
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants