8000 [caffe2] Remove OperatorBase::newstyle_outputs_ by peterbell10 · Pull Request #67093 · pytorch/pytorch · GitHub
[go: up one dir, main page]
Skip to content

[caffe2] Remove OperatorBase::newstyle_outputs_ #67093

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

Closed
wants to merge 22 commits into from
Closed

[caffe2] Remove OperatorBase::newstyle_outputs_ #67093

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
de0edc0
[caffe2] Remove Operator::newstyle_outputs_
peterbell10 Oct 22, 2021
7bff0a1
Update on "[caffe2] Remove Operator::newstyle_outputs_"
peterbell10 Oct 22, 2021
52608de
Update on "[caffe2] Remove Operator::newstyle_outputs_"
peterbell10 Oct 22, 2021
a2bd24e
Update on "[caffe2] Remove Operator::newstyle_outputs_"
peterbell10 Oct 22, 2021
7d7f99f
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Oct 24, 2021
f3ad7c3
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Oct 28, 2021
ae55e45
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Nov 17, 2021
dd3ec09
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Dec 6, 2021
1d690ec
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Jan 3, 2022
fa6a3f6
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Jan 14, 2022
e65e6f3
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Jan 17, 2022
018f0a3
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Jan 18, 2022
8d90b65
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Jan 20, 2022
881db53
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Jan 21, 2022
f66e024
Rebase after master CI break on "[caffe2] Remove OperatorBase::newsty…
peterbell10 Jan 22, 2022
f830c30
Only build IValueInterface when necessary on "[caffe2] Remove Operato…
peterbell10 Feb 14, 2022
2e67734
Rebase and fix merge conflicts on "[caffe2] Remove OperatorBase::news…
peterbell10 Mar 29, 2022
f50c7ca
Rebase on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Apr 8, 2022
b8ae3ac
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Jul 27, 2022
4723019
Rebase on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Oct 10, 2022
3d950c5
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Nov 14, 2022
60eb729
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_"
peterbell10 Jan 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update on "[caffe2] Remove OperatorBase::newstyle_outputs_" 
`OperatorBase` maintains `output_tensors_` and `newstyle_outputs_`
which hold the same list of tensors except one is
`vector<caffe2::Tensor>` and the other is `List<at::Tensor>`.

This instead maintains only `output_tensors_` and handles the
conversions inside of export_caffe2_op_to_c10.

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

[ghstack-poisoned]
  • Loading branch information
@peterbell10
peterbell10 committed Jan 17, 2022
commit e65e6f3ba92f6577a38d1943002d29abc971d91c

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.
0