Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Conversation

malfet
Copy link
Contributor

@malfet malfet commented Oct 14, 2021

As caffe2 builds will be disabled by pytorch/pytorch#66658

facebook-github-bot pushed a commit to pytorch/pytorch that referenced this pull request Oct 22, 2021
Summary:
CAFFE2 has been deprecated for a while, but still included in every PyTorch build.
We should stop building it by default, although CI should still validate that caffe2 code is buildable.

Build even fewer dependencies when compiling mobile builds without Caffe2
Introduce `TEST_CAFFE2` in torch.common.utils
Skip `TestQuantizedEmbeddingOps` and `TestJit.test_old_models_bc`  is code is compiled without Caffe2
Should be landed after pytorch/builder#864

Pull Request resolved: #66658

Reviewed By: driazati, seemethere, janeyx99

Differential Revision: D31669156

Pulled By: malfet

fbshipit-source-id: 1cc45e2d402daf913a4685eb9f841cc3863e458d
@malfet malfet merged commit 66b1e12 into main Oct 22, 2021
@seemethere seemethere deleted the malfet/disable-caffe2-smoke-tests branch October 22, 2021 17:22
malfet added a commit that referenced this pull request Oct 22, 2021
Followup after #864

Hattip to @ngimel for reporting the problem
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants