Skip to content

Conversation

ysiraichi
Copy link
Collaborator

@ysiraichi ysiraichi commented Jun 3, 2021

Stack from ghstack:

This PR makes use of the newly implemented unified at::meta::check_reduction for
validating the inputs and configuring its TensorIterator.

Differential Revision: D29104398

This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jun 3, 2021

💊 CI failures summary and remediations

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


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


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

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

Click here to manually regenerate this comment.

ysiraichi added a commit that referenced this pull request Jun 3, 2021
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

ghstack-source-id: d1a9e9a
Pull Request resolved: #59373
@ysiraichi ysiraichi mentioned this pull request Jun 4, 2021
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

[ghstack-poisoned]
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 8, 2021
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

ghstack-source-id: 0a85628
Pull Request resolved: #59373
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

[ghstack-poisoned]
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 9, 2021
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

ghstack-source-id: 0a85628
Pull Request resolved: #59373
@ysiraichi ysiraichi marked this pull request as ready for review June 9, 2021 13:05
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

[ghstack-poisoned]
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

[ghstack-poisoned]
@ezyang
Copy link
Contributor

ezyang commented Jun 14, 2021

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

@facebook-github-bot
Copy link
Contributor

@ezyang merged this pull request in c078cef.

@facebook-github-bot
Copy link
Contributor

This pull request has been reverted by 81baa7f.

@ysiraichi ysiraichi deleted the gh/ysiraichi/4/head branch June 21, 2021 11:05
ysiraichi added a commit that referenced this pull request Jun 22, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 22, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 22, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 22, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 22, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 22, 2021
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 22, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
facebook-github-bot pushed a commit that referenced this pull request Jun 23, 2021
Summary:
Pull Request resolved: #60360

Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

Test Plan: Imported from OSS

Reviewed By: albanD

Differential Revision: D29265856

Pulled By: ezyang

fbshipit-source-id: 6e9b45ad3fc3852bb142ae2e3d58fc5d0a911aed
facebook-github-bot pushed a commit that referenced this pull request Jun 23, 2021
Summary:
Pull Request resolved: #60361

Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

Test Plan: Imported from OSS

Reviewed By: albanD

Differential Revision: D29265859

Pulled By: ezyang

fbshipit-source-id: 0cca0431569f38a168473b5cc572ced473799961
ezyang added a commit that referenced this pull request Jun 28, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 28, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 28, 2021
…d `torch.any`."


This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 28, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 28, 2021
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 28, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 29, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 29, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 29, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 29, 2021
…d `torch.any`."


This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 29, 2021
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ysiraichi added a commit that referenced this pull request Jun 29, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 29, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 29, 2021
…d `torch.any`."


This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 29, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 29, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 29, 2021
This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Jun 29, 2021
Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

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

[ghstack-poisoned]
facebook-github-bot pushed a commit that referenced this pull request Jun 29, 2021
Summary:
Pull Request resolved: #60362

This PR makes use of the newly implemented unified `at::meta::check_reduction` for
validating the inputs and configuring its `TensorIterator`.

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

Test Plan: Imported from OSS

Reviewed By: albanD

Differential Revision: D29265858

Pulled By: ezyang

fbshipit-source-id: e8961b7da65a31acfed5ac3f5c1f5985ae81ec37
facebook-github-bot pushed a commit that referenced this pull request Jun 29, 2021
Summary:
Pull Request resolved: #60363

Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

Test Plan: Imported from OSS

Reviewed By: albanD

Differential Revision: D29265857

Pulled By: ezyang

fbshipit-source-id: 586914d2aa79028c56988896093945755a2b9781
facebook-github-bot pushed a commit that referenced this pull request Jun 29, 2021
Summary:
Pull Request resolved: #60364

Tracking issue: #55070

This PR was openned so as to solve the CI failures in main when merging: #59371 #59372 #59373 #59937 #59938.

Test Plan: Imported from OSS

Reviewed By: anjali411

Differential Revision: D29265855

Pulled By: ezyang

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

4 participants