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

Deprecate torchtext dependent models and related code. #1831

Closed
wants to merge 2 commits into from

Conversation

xuzhao9
Copy link
Contributor

@xuzhao9 xuzhao9 commented Aug 16, 2023

torchtext is on a much lessen supported model, so we are deprecating the workload and make torchbench not depend on torchtext.

Fixes #1830

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@xuzhao9 merged this pull request in bad4e9a.

@xuzhao9 xuzhao9 deleted the xz9/deprecate-text-models branch August 17, 2023 01:42
facebook-github-bot pushed a commit that referenced this pull request Aug 17, 2023
Summary:
This is a follow-up of #1831

Pull Request resolved: #1833

Reviewed By: janeyx99

Differential Revision: D48432770

Pulled By: xuzhao9

fbshipit-source-id: 2ba8dff02d2ab0df17703427abad126708252762
facebook-github-bot pushed a commit that referenced this pull request Aug 18, 2023
Summary:
This is a follow-up of #1831.

Clip depends on torchmultimodal, which further depends on torchtext:
https://github.com/pytorch/benchmark/actions/runs/5895605535/job/15991723413

Move the clip model to canary_models.

Pull Request resolved: #1837

Reviewed By: msaroufim

Differential Revision: D48480662

Pulled By: xuzhao9

fbshipit-source-id: da57c8aa3df4c520c6afae4342d365b166c8b2dd
xuzhao9 added a commit that referenced this pull request Aug 22, 2023
Summary:
This is a follow-up of #1831

Pull Request resolved: #1833

Reviewed By: janeyx99

Differential Revision: D48432770

Pulled By: xuzhao9

fbshipit-source-id: 2ba8dff02d2ab0df17703427abad126708252762
xuzhao9 added a commit that referenced this pull request Aug 22, 2023
Summary:
torchtext is on a much lessen supported model, so we are deprecating the workload and make torchbench not depend on torchtext.

Fixes #1830

Pull Request resolved: #1831

Reviewed By: abhinavarora

Differential Revision: D48415274

Pulled By: xuzhao9

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

Deprecating models that depend on torchtext
2 participants