-
Notifications
You must be signed in to change notification settings - Fork 96
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
[Bug Report] ttnn.clip interface does not follow Pytorch #14099
Labels
Comments
VirdhatchaniKN
added a commit
that referenced
this issue
Oct 23, 2024
VirdhatchaniKN
added a commit
that referenced
this issue
Oct 23, 2024
VirdhatchaniKN
added a commit
that referenced
this issue
Oct 23, 2024
Hi @kevinwuTT |
@VirdhatchaniKN |
VirdhatchaniKN
added a commit
that referenced
this issue
Oct 25, 2024
Will close issue after PR gets merged : #14127 |
VirdhatchaniKN
added a commit
that referenced
this issue
Oct 25, 2024
VirdhatchaniKN
added a commit
that referenced
this issue
Oct 25, 2024
VirdhatchaniKN
added a commit
that referenced
this issue
Oct 25, 2024
ct-clmsn
pushed a commit
to ct-clmsn/tt-metal
that referenced
this issue
Nov 12, 2024
…torrent#14127) tenstorrent#14099: ttnn.clip interface to follow Pytorch
shwetankTT
added a commit
that referenced
this issue
Nov 20, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
ttnn.clip
has recently changed such that the argumentsmin
andmax
are keywords. However, the equivalent arguments fortorch.clip
and alsotorch.clamp
andHardtanh
are positional instead.https://pytorch.org/docs/stable/generated/torch.clip.html
https://pytorch.org/docs/main/generated/torch.clamp.html
https://pytorch.org/docs/stable/generated/torch.nn.Hardtanh.html#torch.nn.Hardtanh
This changed was observed since
tags/v0.53.0-rc21
but could have been earlier.To Reproduce
Error message:
Expected behavior
If the
min
andmax
args are positional liketorch.clip
thenttnn.clip
would not show any error.Please complete the following environment information:
The text was updated successfully, but these errors were encountered: