Skip to content

Conversation

@kavin-sai-krishna
Copy link
Contributor

This resolves #17842 . This PR provides support for InstanceNorm at the Relax level and fixes the bug in previous TOPI implementation of InstanceNorm.

@kavin-sai-krishna
Copy link
Contributor Author

@tvm-bot rerun

@kavin-sai-krishna
Copy link
Contributor Author

@tlopex Could you please take a look at this PR when you are free. Thanks :)


input_info = [([1, 3, 10, 10], "float32")]

class InstanceNorm2d(Module):
Copy link
Member

Choose a reason for hiding this comment

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

I think we'd better add tests for InstanceNorm1D and InstanceNorm3D since you add it in FX :)

@tlopex tlopex merged commit a1f5a29 into apache:main Jun 23, 2025
10 checks passed
ShiboXing pushed a commit to ShiboXing/tvm that referenced this pull request Aug 10, 2025
…18039)

* add instance norm support

* add support for instance norm

* remove unwanted comments

* remove redundant comments

* remove unused declaration

* Made changes in sematic logic

* fix lint issues

* fix whitespace issues

* Fix whitespace issue

* ffi change in nn.cc

* Fix lint issue

* merge main

* Minor fix
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.

[Bug] InstanceNorm2d with relax not supported

2 participants