Skip to content

Conversation

@bdhirsh
Copy link
Contributor

@bdhirsh bdhirsh commented Nov 3, 2020

See the description in #46399 for all of the details.

That PR broke master due to a linting error and had to be reverted. I generated this PR by reverting the revert, and updating test/test_torch.py to fix the lint errors.

Stack from ghstack:

This reverts commit b3eb0c8.

Differential Revision: D24706531

… the types don't match"

This reverts commit b3eb0c8.

[ghstack-poisoned]
bdhirsh added a commit that referenced this pull request Nov 3, 2020
… the types don't match"

This reverts commit b3eb0c8.

ghstack-source-id: a3f0538
Pull Request resolved: #47288
@bdhirsh bdhirsh changed the title Revert "Revert D24335982: explicitly error out in comparison ops when the types don't match" Fix type promotion + broadcasting bug Nov 3, 2020
@bdhirsh bdhirsh requested a review from mruberry November 3, 2020 16:55
Copy link
Collaborator

@mruberry mruberry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stamped!

@codecov
Copy link

codecov bot commented Nov 3, 2020

Codecov Report

Merging #47288 into gh/bdhirsh/31/base will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@                  Coverage Diff                   @@
##           gh/bdhirsh/31/base   #47288      +/-   ##
======================================================
- Coverage               60.81%   60.81%   -0.01%     
======================================================
  Files                    2749     2749              
  Lines                  254098   254108      +10     
======================================================
+ Hits                   154537   154539       +2     
- Misses                  99561    99569       +8     

@facebook-github-bot
Copy link
Contributor

@bdhirsh merged this pull request in fe17269.

@facebook-github-bot facebook-github-bot deleted the gh/bdhirsh/31/head branch November 8, 2020 15:18
@gchanan gchanan added the module: bc-breaking Related to a BC-breaking change label Nov 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged module: bc-breaking Related to a BC-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants