Skip to content

Conversation

krshrimali
Copy link
Contributor

@krshrimali krshrimali commented Jul 23, 2021

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jul 23, 2021

🔗 Helpful links

💊 CI failures summary and remediations

As of commit 714cc3d (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.

@krshrimali krshrimali requested a review from mruberry July 23, 2021 05:25
@krshrimali krshrimali added the module: testing Issues related to the torch.testing module (not tests) label Jul 23, 2021
@jbschlosser jbschlosser added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Jul 23, 2021
@mruberry mruberry requested a review from zou3519 July 30, 2021 06:37
(()),
((S, )),
((S, S)),
((S, M, S))
Copy link
Collaborator

Choose a reason for hiding this comment

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

We probably don't even need this case

@@ -6310,6 +6326,13 @@ def gradcheck_wrapper_triangular_input(op, input, *args, upper=False, **kwargs):
dtypes=all_types_and(torch.float16, torch.bfloat16, torch.bool),
supports_forward_ad=True,
sample_inputs_func=sample_inputs_max_min_binary,),
OpInfo('nn.functional.relu',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice!

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.

Awesome! Thanks @krshrimali.

@zou3519 would you like to check this, make sure it has the sample inputs you need, and shepherd it through?

@facebook-github-bot
Copy link
Contributor

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

@zou3519
Copy link
Contributor

zou3519 commented Jul 30, 2021

Yup, can do!

@facebook-github-bot
Copy link
Contributor

@zou3519 merged this pull request in a705b8f.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla signed Merged module: testing Issues related to the torch.testing module (not tests) open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants