8000 [caffe2] Export operators to c10 without including ATen/Tensor.h by peterbell10 · Pull Request #67096 · pytorch/pytorch · GitHub
[go: up one dir, main page]
Skip to content

[caffe2] Export operators to c10 without including ATen/Tensor.h #67096

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 27 commits into from
Closed

[caffe2] Export operators to c10 without including ATen/Tensor.h #67096

wants to merge 27 commits into from

Conversation

peterbell10
Copy link
Collaborator
@peterbell10 peterbell10 commented Oct 22, 2021 •
edited
Loading

Stack from ghstack (oldest at bottom):

With this change, the only caffe2 files that depend on ATen/Tensor.h
are ones that directly use the ATen API. Specifically,

[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]

Differential Revision: D32289810

@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
1cffd06 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

[ghstack-poisoned]
This was referenced Oct 22, 2021
Closed
Closed
@pytorch-probot
Copy link
pytorch-probot bot commented Oct 22, 2021 •
edited by pytorch-bot bot
Loading
CI Flow Status

⚛️ CI Flow

Ruleset - Version: v1
Ruleset - File: https://github.com/pytorch/pytorch/blob/54813608420773fbb3ba1b0b6d418d102eccbd2e/.github/generated-ciflow-ruleset.json
PR ciflow labels: ciflow/default,ciflow/cpu

10000
Workflows Labels (bold enabled) Status
Triggered Workflows
caffe2-linux-xenial-py3.7-gcc5.4 ciflow/all, ciflow/cpu, ciflow/linux, ciflow/trunk ✅ triggered
linux-binary-conda ciflow/binaries, ciflow/binaries_conda, ciflow/default ✅ triggered
linux-binary-libtorch-cxx11-abi ciflow/binaries, ciflow/binaries_libtorch, ciflow/default ✅ triggered
linux-binary-libtorch-pre-cxx11 ciflow/binaries, ciflow/binaries_libtorch, ciflow/default ✅ triggered
linux-binary-manywheel ciflow/binaries, ciflow/binaries_wheel, ciflow/default ✅ triggered
linux-bionic-py3.7-clang9 ciflow/all, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/noarch, ciflow/trunk, ciflow/xla ✅ triggered
linux-docs ciflow/all, ciflow/cpu, ciflow/default, ciflow/docs, ciflow/linux, ciflow/trunk ✅ triggered
linux-docs-push ciflow/all, ciflow/cpu, ciflow/linux, ciflow/scheduled ✅ triggered
linux-vulkan-bionic-py3.7-clang9 ciflow/all, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/trunk, ciflow/vulkan ✅ triggered
linux-xenial-cuda11.3-py3.7-gcc7 ciflow/all, ciflow/cuda, ciflow/default, ciflow/linux, ciflow/trunk ✅ triggered
linux-xenial-cuda11.3-py3.7-gcc7-bazel-test ciflow/all, ciflow/bazel, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/trunk ✅ triggered
linux-xenial-py3-clang5-mobile-build ciflow/all, ciflow/default, ciflow/linux, ciflow/mobile, ciflow/trunk ✅ triggered
linux-xenial-py3-clang5-mobile-custom-build-static ciflow/all, ciflow/default, ciflow/linux, ciflow/mobile, ciflow/trunk ✅ triggered
linux-xenial-py3.7-clang7-asan ciflow/all, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/sanitizers, ciflow/trunk ✅ triggered
linux-xenial-py3.7-clang7-onnx ciflow/all, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/onnx, ciflow/trunk ✅ triggered
linux-xenial-py3.7-gcc5.4 ciflow/all, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/trunk ✅ triggered
linux-xenial-py3.7-gcc7 ciflow/all, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/trunk ✅ triggered
linux-xenial-py3.7-gcc7-no-ops ciflow/all, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/trunk ✅ triggered
parallelnative-linux-xenial-py3.7-gcc5.4 ciflow/all, ciflow/cpu, ciflow/linux, ciflow/trunk ✅ triggered
pytorch-linux-xenial-py3-clang5-android-ndk-r19c-build ciflow/all, ciflow/android, ciflow/cpu, ciflow/linux, ciflow/trunk ✅ triggered
pytorch-linux-xenial-py3-clang5-android-ndk-r19c-gradle-custom-build-single ciflow/all, ciflow/android, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/trunk ✅ triggered
pytorch-linux-xenial-py3-clang5-android-ndk-r19c-gradle-custom-build-single-full-jit ciflow/all, ciflow/android, ciflow/cpu, ciflow/default, ciflow/linux, ciflow/trunk ✅ triggered
win-vs2019-cpu-py3 ciflow/all, ciflow/cpu, ciflow/default, ciflow/trunk, ciflow/win ✅ triggered
win-vs2019-cuda11.3-py3 ciflow/all, ciflow/cuda, ciflow/default, ciflow/trunk, ciflow/win ✅ triggered
Skipped Workflows
docker-builds ciflow/all, ciflow/trunk 🚫 skipped
ios-12-5-1-arm64 ciflow/all, ciflow/ios, ciflow/macos, ciflow/trunk 🚫 skipped
ios-12-5-1-arm64-coreml ciflow/all, ciflow/ios, ciflow/macos, ciflow/trunk 🚫 skipped
ios-12-5-1-arm64-custom-ops ciflow/all, ciflow/ios, ciflow/macos, ciflow/trunk 🚫 skipped
ios-12-5-1-arm64-full-jit ciflow/all, ciflow/ios, ciflow/macos, ciflow/trunk 🚫 skipped
ios-12-5-1-arm64-metal ciflow/all, ciflow/ios, ciflow/macos, ciflow/trunk 🚫 skipped
ios-12-5-1-x86-64 ciflow/all, ciflow/ios, ciflow/macos, ciflow/trunk 🚫 skipped
ios-12-5-1-x86-64-coreml ciflow/all, ciflow/ios, ciflow/macos, ciflow/trunk 🚫 skipped
ios-12-5-1-x86-64-full-jit ciflow/all, ciflow/ios, ciflow/macos, ciflow/trunk 🚫 skipped
libtorch-linux-xenial-cuda10.2-py3.7-gcc7 ciflow/all, ciflow/cuda, ciflow/libtorch, ciflow/linux, ciflow/trunk 🚫 skipped
libtorch-linux-xenial-cuda11.3-py3.7-gcc7 ciflow/all, ciflow/cuda, ciflow/libtorch, ciflow/linux, ciflow/trunk 🚫 skipped
linux-bionic-cuda10.2-py3.9-gcc7 ciflow/all, ciflow/cuda, ciflow/linux, ciflow/slow, ciflow/trunk 🚫 skipped
linux-bionic-rocm4.5-py3.7 ciflow/all, ciflow/linux, ciflow/rocm, ciflow/trunk 🚫 skipped
linux-xenial-cuda11.3-py3.7-gcc7-no-ops ciflow/all, ciflow/cuda, ciflow/linux, ciflow/trunk 🚫 skipped
macos-10-15-py3-arm64 ciflow/all, ciflow/macos, ciflow/trunk 🚫 skipped
macos-10-15-py3-lite-interpreter-x86-64 ciflow/all, ciflow/macos, ciflow/trunk 🚫 skipped
macos-11-py3-x86-64 ciflow/all, ciflow/macos, ciflow/trunk 🚫 skipped
periodic-libtorch-linux-bionic-cuda11.5-py3.7-gcc7 ciflow/all, ciflow/cuda, ciflow/libtorch, ciflow/linux, ciflow/scheduled 🚫 skipped
periodic-libtorch-linux-xenial-cuda11.1-py3.7-gcc7 ciflow/all, ciflow/cuda, ciflow/libtorch, ciflow/linux, ciflow/scheduled 🚫 skipped
periodic-linux-bionic-cuda11.5-py3.7-gcc7 ciflow/all, ciflow/cuda, ciflow/linux, ciflow/scheduled 🚫 skipped
periodic-linux-xenial-cuda10.2-py3-gcc7-slow-gradcheck ciflow/all, ciflow/cuda, ciflow/linux, ciflow/scheduled, ciflow/slow, ciflow/slow-gradcheck 🚫 skipped
periodic-linux-xenial-cuda11.1-py3.7-gcc7-debug ciflow/all, ciflow/cuda, ciflow/linux, ciflow/scheduled 🚫 skipped
periodic-win-vs2019-cuda11.1-py3 ciflow/all, ciflow/cuda, ciflow/scheduled, ciflow/win 🚫 skipped
periodic-win-vs2019-cuda11.5-py3 ciflow/all, ciflow/cuda, ciflow/scheduled, ciflow/win 🚫 skipped

You can add a comment to the PR and tag @pytorchbot with the following commands: 
# ciflow rerun, "ciflow/default" will always be added automatically
@pytorchbot ciflow rerun

# ciflow rerun with additional labels "-l <ciflow/label_name>", which is equivalent to adding these labels manually and trigger the rerun
@pytorchbot ciflow rerun -l ciflow/scheduled -l ciflow/slow

For more information, please take a look at the CI Flow Wiki.

@peterbell10 peterbell10 mentioned this pull request Oct 22, 2021
Closed
peterbell10 added a commit that referenced this pull request Oct 22, 2021
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
da06b0a 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: 990ba65
Pull Request resolved: #67096
@facebook-github-bot
Copy link
Contributor
facebook-github-bot commented Oct 22, 2021 •
edited
Loading

🔗 Helpful links

✅ No Failures (0 Pending)

As of commit d53ecbb (more details on the Dr. CI page):

Expand to see more

💚 💚 Looks good so far! There are no failures yet. 💚 💚

This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
93b8851 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

[ghstack-poisoned]
@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
acd625b 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Oct 22, 2021
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
c531b63 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: dfc63c0
Pull Request resolved: #67096
@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
0a40a8b 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

[ghstack-poisoned]
@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
10000
b8d816e 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Oct 24, 2021
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
26afb04 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: 7e9460d
Pull Request resolved: #67096
@peterbell10 peterbell10 marked this pull request as ready for review October 25, 2021 13:14
@peterbell10 peterbell10 requested review from ezyang and dagitses and removed request for ezyang October 25, 2021 13:16
@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
9d6cd26 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

[ghstack-poisoned]
@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
63b20e7 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Oct 28, 2021
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
b273a5a 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: 67276a5
Pull Request resolved: #67096
@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
7a2321b 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Oct 29, 2021
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
c40d87c 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: 3862e06
Pull Request resolved: #67096
@dagitses
Copy link
Collaborator
dagitses commented Nov 9, 2021

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

@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
d53ecbb 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

Differential Revision: [D32289810](https://our.internmc.facebook.com/intern/diff/D32289810)

[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Jul 27, 2022
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
acc1a8c 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: ca55ff6
Pull Request resolved: #67096
@github-actions
Copy link
Contributor

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

@github-actions github-actions bot added the Stale label Sep 25, 2022
@peterbell10 peterbell10 removed the Stale label Sep 25, 2022
@facebook-github-bot
Copy link
Contributor

/easycla

As part of the transition to the PyTorch Foundation, this project now requires contributions be covered under the new CLA. See #85559 for additional details.

This comment will trigger a new check of this PR. If you are already covered, you will simply see a new "EasyCLA" check that passes. If you are not covered, a bot will leave a new comment with a link to sign.

@linux-foundation-easycla
Copy link
linux-foundation-easycla bot commented Oct 4, 2022 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@peterbell10
Rebase on "[caffe2] Export operators to c10 without including ATen/Te…
4fa5b6a 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

Differential Revision: [D32289810](https://our.internmc.facebook.com/intern/diff/D32289810)

[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Oct 10, 2022
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
be24cbe 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: 4c81e7f
Pull Request resolved: #67096
@pytorch-bot
Copy link
pytorch-bot bot commented Oct 10, 2022 •
edited
Loading

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/67096

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 214e17c:
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@peterbell10 peterbell10 added the ciflow/trunk Trigger trunk jobs on your pull request label Oct 10, 2022
@dagitses
Copy link
Collaborator

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

@kit1980
Copy link
Contributor
kit1980 commented Oct 25, 2022

@peterbell10 @dagitses is this something we want to merge now?

@peterbell10
Copy link
Collaborator Author
peterbell10 commented Oct 30, 2022 •
edited
Loading

There is still a caffe2 job in PyTorch trunk CI so I think it still has value.

@kit1980
Copy link
Contributor
kit1980 commented Nov 14, 2022

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a rebase job. Check the current status here

@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
68342d7 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

Differential Revision: [D32289810](https://our.internmc.facebook.com/intern/diff/D32289810)

[ghstack-poisoned]
@pytorchmergebot
Copy link
Collaborator

Successfully rebased gh/peterbell10/189/orig onto refs/remotes/origin/viable/strict, please pull locally before adding more changes (for example, via ghstack checkout https://github.com/pytorch/pytorch/pull/67096)

pytorchmergebot pushed a commit that referenced this pull request Nov 14, 2022
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
a299db5 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: 4ed48bb
Pull Request resolved: #67096
@kit1980 kit1980 added the topic: not user facing topic category label Nov 14, 2022
@github-actions
Copy link
Contributor

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

@github-actions github-actions bot added the Stale label Jan 14, 2023
peterbell10 added a commit to peterbell10/pytorch that referenced this pull request Jan 23, 2023
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
8832434 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: 4ed48bb
Pull Request resolved: pytorch#67096
@peterbell10
Update on "[caffe2] Export operators to c10 without including ATen/Te…
214e17c 
…nsor.h"

With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

Differential Revision: [D32289810](https://our.internmc.facebook.com/intern/diff/D32289810)

[ghstack-poisoned]
peterbell10 added a commit that referenced this pull request Jan 23, 2023
@peterbell10
[caffe2] Export operators to c10 without including ATen/Tensor.h
90b0de7 
With this change, the only caffe2 files that depend on `ATen/Tensor.h`
are ones that directly use the ATen API. Specifically,
```
[
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/contrib/aten/aten_op_gpu.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/tensor.cc.o",
  "caffe2/CMakeFiles/torch_cuda_cpp.dir/operators/layer_norm_op.cu.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/IValueInterface.cc.o",
  "caffe2/CMakeFiles/cuda_tensor_interop_test.dir/__/aten/src/ATen/test/cuda_tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/torch_cpu.dir/contrib/aten/aten_op.cc.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state_gpu.dir/python/pybind_state.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/layer_norm_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_c10_op_to_caffe2.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/export_caffe2_op_to_c10.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/operators/enforce_finite_op.cc.o",
  "caffe2/CMakeFiles/torch_cpu.dir/core/operator.cc.o",
  "caffe2/CMakeFiles/tensor_interop_test.dir/__/aten/src/ATen/test/tensor_interop_test.cpp.o",
  "caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o"
]
```

ghstack-source-id: f183da6
Pull Request resolved: #67096
@peterbell10 peterbell10 requested a review from kit1980 January 24, 2023 11:44
@peterbell10 peterbell10 closed this Mar 4, 2024
@github-actions github-actions bot deleted the gh/peterbell10/189/head branch April 4, 2024 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dagitses dagitses dagitses approved these changes

@kit1980 kit1980 Awaiting requested review from kit1980

Assignees
No one assigned
Labels
ciflow/trunk Trigger trunk jobs on your pull request cla signed no-stale open source topic: not user facing topic category
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@peterbell10 @facebook-github-bot @dagitses @kit1980 @pytorchmergebot @ezyang @suo @pytorchbot
0