Skip to content

Conversation

@mruberry
Copy link
Collaborator

@mruberry mruberry commented Apr 3, 2020

Adds test for behavior reported in #35257 to ensure it doesn't regress. The test was extended to reveal three additional issues:

@mruberry mruberry added the module: numpy Related to numpy support, and also numpy compatibility of our operators label Apr 3, 2020
@dr-ci
Copy link

dr-ci bot commented Apr 3, 2020

💊 Build failures summary and remediations

As of commit fe2f015 (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 on the GitHub issue tracker.

See how this bot performed.

This comment has been revised 16 times.

@mruberry mruberry changed the title Adds NumPy scalar interaction test Adds NumPy array x Torch tensor binary ufunc interaction test Apr 10, 2020
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

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

@facebook-github-bot
Copy link
Contributor

@mruberry merged this pull request in 254be6a.

ashishfarmer pushed a commit to ashishfarmer/pytorch that referenced this pull request Apr 13, 2020
…h#35945)

Summary:
Adds test for behavior reported in pytorch#35257 to ensure it doesn't regress. The test was extended to reveal three additional issues:

- pytorch#36363
- pytorch#36058
- pytorch#36057
Pull Request resolved: pytorch#35945

Differential Revision: D20984429

Pulled By: mruberry

fbshipit-source-id: a15be9455afba9c77e40c337a860f9be348bf8d5
@mruberry mruberry deleted the np_scalar_interaction branch June 18, 2020 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: numpy Related to numpy support, and also numpy compatibility of our operators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants