Skip to content

↔ [Converter] Add support for aten::atan2 in Torch-TensorRT #938

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
Njuapp opened this issue Mar 23, 2022 · 3 comments · Fixed by #1381
Closed

↔ [Converter] Add support for aten::atan2 in Torch-TensorRT #938

Njuapp opened this issue Mar 23, 2022 · 3 comments · Fixed by #1381
Assignees
Labels
component: converters Issues re: Specific op converters component: partitioning feature request New feature or request No Activity

Comments

@Njuapp
Copy link
Contributor

Njuapp commented Mar 23, 2022

Unsupported operator: aten::atan2(Tensor self, Tensor other) -> (Tensor)

  • Function Schema:
    aten::atan2(Tensor self, Tensor other) -> (Tensor)

  • Original PyTorch API:
    torch.atan2(input, other, *, out=None) → [Tensor]

  • Relevant TensorRT Documentation:
    None

Alternatives

Additional context

When using Fourier transformation in audio segmentation models, torch.atan2 will be used.
Because variable audio input lengths requires dynamic shape, fallback to pytorch would cause runtime error when creating example inputs by partitioning::generateRandomInputs.
It reports

RuntimeError: Trying to create tensor with negative dimension -1: [1, 1, -1]

Therefore, we need convertor for torch.atan2 to successfully convert such models.

@Njuapp Njuapp added component: converters Issues re: Specific op converters feature request New feature or request labels Mar 23, 2022
@narendasan
Copy link
Collaborator

cc: @ncomly-nvidia

@narendasan
Copy link
Collaborator

cc: @apbose

@github-actions
Copy link

github-actions bot commented Oct 5, 2022

This issue has not seen activity for 90 days, Remove stale label or comment or this will be closed in 10 days

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: converters Issues re: Specific op converters component: partitioning feature request New feature or request No Activity
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants