Skip to content

Conversation

apbose
Copy link
Collaborator

@apbose apbose commented Jun 18, 2024

No description provided.

@github-actions github-actions bot added the component: core Issues re: The core compiler label Jun 18, 2024
@github-actions github-actions bot requested a review from narendasan June 18, 2024 22:33
@HolyWu
Copy link
Contributor

HolyWu commented Jun 26, 2024

@apbose
Copy link
Collaborator Author

apbose commented Jun 26, 2024

Thanks @HolyWu for pointing out the above. @narendasan would the changes in PR https://github.com/pytorch/TensorRT/pull/2855/files#diff-97c67f0cea25ec7aa88ca61594d9ad62e9fb575cab3647b079cb718f81308454 work or would we have to create a type map for kINT32 to kINT64 in the torchscript path?

@narendasan
Copy link
Collaborator

Yeah neither of these PRs implements the feature fully since we need to either amend truncation in TorchScript or have a TorchScript specific type map

@narendasan
Copy link
Collaborator

@apbose going to close this. reopen a new pr with the comprehensive changes if its still needed

@narendasan narendasan closed this Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants