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

Default to supported model batch size instead of metadata if batch size not specified #2015

Closed
wants to merge 3 commits into from

Conversation

eellison
Copy link
Contributor

@eellison eellison commented Oct 26, 2023

Stack from ghstack (oldest at bottom):

This was breaking stable_diffusion_unet.

Differential Revision: D50747937

eellison added a commit that referenced this pull request Oct 26, 2023
…ze not specified

ghstack-source-id: dacb21b05bb4ffd664f957a18e0060c426988330
Pull Request resolved: #2015
torchbenchmark/util/model.py Outdated Show resolved Hide resolved
…if batch size not specified"


This was breaking stable_diffusion_unet.


[ghstack-poisoned]
eellison added a commit that referenced this pull request Oct 27, 2023
…ze not specified

ghstack-source-id: 9650512d1875e64cdbbe53a8259f3d9b9cbe2674
Pull Request resolved: #2015
@eellison eellison temporarily deployed to docker-s3-upload October 27, 2023 01:13 — with GitHub Actions Inactive
@eellison eellison temporarily deployed to docker-s3-upload October 27, 2023 01:13 — with GitHub Actions Inactive
@eellison
Copy link
Contributor Author

@pytorchbot merge

…if batch size not specified"


This was breaking stable_diffusion_unet.


[ghstack-poisoned]
eellison added a commit that referenced this pull request Oct 27, 2023
…ze not specified

ghstack-source-id: 037c67ddab28909e96d395f87bac632cc664b7c8
Pull Request resolved: #2015
@eellison eellison temporarily deployed to docker-s3-upload October 27, 2023 17:35 — with GitHub Actions Inactive
@eellison eellison temporarily deployed to docker-s3-upload October 27, 2023 17:35 — with GitHub Actions Inactive
@xuzhao9
Copy link
Contributor

xuzhao9 commented Oct 27, 2023

@eellison Can you help import this PR to fbcode so that we can land it? Sorry but pytorch/benchmark does not use GH first for now.

@eellison
Copy link
Contributor Author

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

@eellison
Copy link
Contributor Author

imported !

@facebook-github-bot
Copy link
Contributor

@eellison merged this pull request in f235144.

@facebook-github-bot facebook-github-bot deleted the gh/eellison/2/head branch October 31, 2023 14:22
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.

5 participants