-
Notifications
You must be signed in to change notification settings - Fork 351
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
full_like to full decomposition moving to decomposition.py for dynami… #3289
base: main
Are you sure you want to change the base?
Conversation
e0cd635
to
5ff1beb
Compare
@apbose do you have a test case? |
@apbose I see your comment : #3140 (comment). Can you provide more context on why this change is required ? |
@narendasan the test case already exists in https://github.com/pytorch/TensorRT/blob/main/tests/py/dynamo/lowering/test_decompositions.py#L424 |
@peri044 I removed the
|
5ff1beb
to
5aca1dd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some changes that do not conform to Python style guidelines:
--- /home/runner/work/TensorRT/TensorRT/py/torch_tensorrt/dynamo/lowering/_decompositions.py 2024-11-19 20:01:46.728609+00:00
+++ /home/runner/work/TensorRT/TensorRT/py/torch_tensorrt/dynamo/lowering/_decompositions.py 2024-11-19 20:02:09.200571+00:00
@@ -416,12 +416,12 @@
return torch.nn.functional.group_norm(input, input.shape[1], weight, bias, eps)
else:
return torch.nn.functional.batch_norm(
input, running_mean, running_var, weight, bias, False, momentum, eps
)
-
-
+
+
@register_torch_trt_decomposition(
torch.ops.aten.full_like, registry=TORCH_TRT_DECOMPOSITIONS
)
def full_like_decomposition(*args, **kwargs) -> torch.Tensor:
input = args[0]
5aca1dd
to
b73bb83
Compare
Oh the PR now is failing since the graph post lowering is an empty one
Modifying the test now to make it non empty |
b73bb83
to
1965a87
Compare
No description provided.