Skip to content

Conversation

@Deivanayaki-S
Copy link
Contributor

This PR adds support for the torch.outer operation in exported program and fx graph translator torch frontend. By adding this support, the Sentence similarity nomic-ai/nomic-embed-text-v1.5 model is now able to run successfully:

@Deivanayaki-S Deivanayaki-S marked this pull request as ready for review May 9, 2025 04:59
@Deivanayaki-S
Copy link
Contributor Author

cc @Hzfengsy @tlopex

Copy link
Member

@Hzfengsy Hzfengsy left a comment

Choose a reason for hiding this comment

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

Overall LGTM

R.output(gv)
return gv

input_infos = [
Copy link
Member

Choose a reason for hiding this comment

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

Better to update the format :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Hzfengsy Lint CI check failed when this format was not followed

Comment on lines 894 to 907
input_infos = [
(
[
3,
],
"float32",
),
(
[
4,
],
"float32",
),
]
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
input_infos = [
(
[
3,
],
"float32",
),
(
[
4,
],
"float32",
),
]
input_infos = [([3], "float32"), ([4], "float32")]

Copy link
Member

Choose a reason for hiding this comment

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

Try this one :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Hzfengsy Thanks! I've updated the format, and the lint check has passed successfully :)

@Hzfengsy Hzfengsy merged commit f4704f2 into apache:main May 10, 2025
10 checks passed
ShiboXing pushed a commit to ShiboXing/tvm that referenced this pull request Aug 10, 2025
…FX graph (apache#17930)

* add torch.outer op support into exported program and fx translator

* fix lint issues

* fix cpp lints

* update the format of input info n fx test script

---------

Co-authored-by: deivanayakisankaralingam <deiva@Deivanayaki>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants