Skip to content
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.

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

[Static Runtime] Support clamp.Tensor #58191

Closed
wants to merge 1 commit into from
Closed

Conversation

@hlu1
Copy link
Contributor

@hlu1 hlu1 commented May 12, 2021

Summary: There are two clamp overloads: clamp.Scalar and clamp.Tensor. SR needs to support both or has checks in place to avoid runtime errors. Supporting both is not too hard so here we are.

Differential Revision: D28371949

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot facebook-github-bot commented May 12, 2021

💊 CI failures summary and remediations

As of commit 0d40bcf (more details on the Dr. CI page):


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-scanned failure(s)

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.

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot facebook-github-bot commented May 12, 2021

This pull request was exported from Phabricator. Differential Revision: D28371949

Summary:
Pull Request resolved: pytorch#58191

There are two clamp overloads: clamp.Scalar and clamp.Tensor. SR needs to support both or has checks in place to avoid runtime errors. Supporting both is not too hard so here we are.

Reviewed By: edvgha

Differential Revision: D28371949

fbshipit-source-id: 430c6ba29655e8d331aee3ed3d2b6ed7ec988662
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot facebook-github-bot commented May 13, 2021

This pull request was exported from Phabricator. Differential Revision: D28371949

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot facebook-github-bot commented May 14, 2021

This pull request has been merged in 993a35a.

krshrimali pushed a commit to krshrimali/pytorch that referenced this issue May 19, 2021
Summary:
Pull Request resolved: pytorch#58191

There are two clamp overloads: clamp.Scalar and clamp.Tensor. SR needs to support both or has checks in place to avoid runtime errors. Supporting both is not too hard so here we are.

Reviewed By: edvgha

Differential Revision: D28371949

fbshipit-source-id: 0ec6b8a0b8c6277e50d8e51e4e7a45aa62211e22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants