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

Skip to content

add __deepcopy__ back to Parameter#12886

Closed
weiyangfb wants to merge 2 commits into
pytorch:masterfrom
weiyangfb:copy_parameter
Closed

add __deepcopy__ back to Parameter#12886
weiyangfb wants to merge 2 commits into
pytorch:masterfrom
weiyangfb:copy_parameter

Conversation

@weiyangfb
Copy link
Copy Markdown
Contributor

Comment thread test/test_torch.py Outdated

This comment was marked as off-topic.

Comment thread torch/nn/parameter.py Outdated

This comment was marked as off-topic.

This comment was marked as off-topic.

@weiyangfb
Copy link
Copy Markdown
Contributor Author

is this ready to go? @apaszke @soumith

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

soumith is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

weiyangfb has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@ezyang ezyang added the merged label Jun 25, 2019
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
- fix pytorch#315
- add `__deepcopy__` back to Parameter class
Pull Request resolved: pytorch#12886

Differential Revision: D12838771

Pulled By: weiyangfb

fbshipit-source-id: b2ce12244e36f981d89f6c7cdead63237dd820ea
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.

copy of Parameter converts it to Variable

5 participants