Skip to content

full_like evaluator #2917

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

Closed
wants to merge 1 commit into from
Closed

full_like evaluator #2917

wants to merge 1 commit into from

Conversation

apbose
Copy link
Collaborator

@apbose apbose commented Jun 13, 2024

No description provided.

@github-actions github-actions bot added component: tests Issues re: Tests component: conversion Issues re: Conversion stage component: api [Python] Issues re: Python API component: dynamo Issues relating to the `torch.compile` or `torch._dynamo.export` paths labels Jun 13, 2024
@github-actions github-actions bot requested a review from peri044 June 13, 2024 19:33
Comment on lines 20 to +21
from torch_tensorrt.fx.types import TRTTensor
from torch_tensorrt.fx.utils import Frameworks, unified_dtype_converter
Copy link
Collaborator

Choose a reason for hiding this comment

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

We have our own types and switch from unified_dtype_converter to using _enums.dtype.to and _enums.dtype.from for type conversions. We shouldn't be using FX utils anymore.

full_like_np_tensor = None
tensor_shape = args[0].shape
if kwargs.get("dtype") is not None:
full_like_np_tensor = np.full(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you also add dynamic shape support ? what if the input shape has -1 in it ? Please refer to fill layer which can help you out here.

@peri044
Copy link
Collaborator

peri044 commented Aug 6, 2024

@apbose what is the status of full_like ?

@apbose
Copy link
Collaborator Author

apbose commented Aug 6, 2024

I did not work on this since long. I should be able to take a look at this next week when I am done with dynamic shape of aten::chunk and aten::tile and merge the remaining PRs.

@peri044
Copy link
Collaborator

peri044 commented Aug 22, 2024

Closing this as we now have a lowering pass that handles this.

@peri044 peri044 closed this Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla signed component: api [Python] Issues re: Python API component: conversion Issues re: Conversion stage component: dynamo Issues relating to the `torch.compile` or `torch._dynamo.export` paths component: tests Issues re: Tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants