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

Fixes to prep for weights_only default flip #2514

Closed
wants to merge 1 commit into from

Conversation

mikaylagawarecki
Copy link
Contributor

@mikaylagawarecki mikaylagawarecki commented Oct 17, 2024

Some fixes for pytorch/pytorch#137602

@xuzhao9
Copy link
Contributor

xuzhao9 commented Oct 18, 2024

cc @kit1980, can you help review this?

@@ -29,7 +31,13 @@ def __init__(self, test, device, batch_size=None, extra_args=[]):
)

self.model = models.resnet18(num_classes=10)
prev_wo_envvar = os.environ.get("TORCH_FORCE_NO_WEIGHTS_ONLY_LOAD", None)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We don't have access to the torch.load call within opacus here, updating opacus also won't help because torchbench doesn't depend on a nightly of opacus but rather stable release

@facebook-github-bot
Copy link
Contributor

@kit1980 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 21, 2024
Summary:
Some fixes for pytorch/pytorch#137602


Reviewed By: xuzhao9

Differential Revision: D64628614

Pulled By: mikaylagawarecki
@facebook-github-bot
Copy link
Contributor

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

@mikaylagawarecki mikaylagawarecki marked this pull request as ready for review October 21, 2024 22:49
mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 21, 2024
Summary:
Some fixes for pytorch/pytorch#137602


Reviewed By: xuzhao9

Differential Revision: D64628614

Pulled By: mikaylagawarecki
@facebook-github-bot
Copy link
Contributor

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

Summary:
Some fixes for pytorch/pytorch#137602


Reviewed By: xuzhao9

Differential Revision: D64628614

Pulled By: mikaylagawarecki
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@mikaylagawarecki merged this pull request in fb590d9.

mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 22, 2024
mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 22, 2024
Summary: Pull Request resolved: pytorch#2524

Differential Revision: D64771621

Pulled By: mikaylagawarecki
mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 24, 2024
Summary:
Some fixes for pytorch/pytorch#137602

Pull Request resolved: pytorch#2514

Reviewed By: xuzhao9

Differential Revision: D64628614

Pulled By: mikaylagawarecki

fbshipit-source-id: edebf25cc6648919d5673a3baeaffdac26e5b91f
mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 24, 2024
Summary: Pull Request resolved: pytorch#2524

Differential Revision: D64771621

Pulled By: mikaylagawarecki
mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 24, 2024
Summary: Pull Request resolved: pytorch#2524

Differential Revision: D64771621

Pulled By: mikaylagawarecki
mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 24, 2024
Summary: Pull Request resolved: pytorch#2524

Differential Revision: D64771621

Pulled By: mikaylagawarecki
facebook-github-bot pushed a commit that referenced this pull request Oct 25, 2024
Summary: Pull Request resolved: #2524

Reviewed By: kit1980

Differential Revision: D64771621

Pulled By: mikaylagawarecki

fbshipit-source-id: 545f3d528cfbe2668c8d37e98e99423cd77a8e8e
mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 25, 2024
Summary:
Some fixes for pytorch/pytorch#137602

Pull Request resolved: pytorch#2514

Reviewed By: xuzhao9

Differential Revision: D64628614

Pulled By: mikaylagawarecki

fbshipit-source-id: edebf25cc6648919d5673a3baeaffdac26e5b91f
mikaylagawarecki added a commit to mikaylagawarecki/benchmark that referenced this pull request Oct 25, 2024
Summary: Pull Request resolved: pytorch#2524

Reviewed By: kit1980

Differential Revision: D64771621

Pulled By: mikaylagawarecki

fbshipit-source-id: 545f3d528cfbe2668c8d37e98e99423cd77a8e8e
mikaylagawarecki added a commit that referenced this pull request Oct 25, 2024
Summary:
Some fixes for pytorch/pytorch#137602

Pull Request resolved: #2514

Reviewed By: xuzhao9

Differential Revision: D64628614

Pulled By: mikaylagawarecki

fbshipit-source-id: edebf25cc6648919d5673a3baeaffdac26e5b91f
mikaylagawarecki added a commit that referenced this pull request Oct 25, 2024
Summary: Pull Request resolved: #2524

Reviewed By: kit1980

Differential Revision: D64771621

Pulled By: mikaylagawarecki

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

3 participants