Skip to content

[Hexagon] Add test for depthwise conv2d schedule#11138

Merged
mehrdadh merged 5 commits intoapache:mainfrom
farshidsp:farshidsp/hexagon/add_operator_tests
Apr 28, 2022
Merged

[Hexagon] Add test for depthwise conv2d schedule#11138
mehrdadh merged 5 commits intoapache:mainfrom
farshidsp:farshidsp/hexagon/add_operator_tests

Conversation

@farshidsp
Copy link
Copy Markdown
Contributor

This PR adds depth wise conv2d tests on hexagon target.
cc @mehrdadh @csullivan @Lunderberg

@github-actions github-actions Bot requested a review from mehrdadh April 27, 2022 00:51
@mehrdadh mehrdadh changed the title [Hexagon] Add test for registered schedules [Hexagon] Add test for depthwise conv2d schedule Apr 27, 2022
@mehrdadh
Copy link
Copy Markdown
Member

This PR could fail due to long hexagon CI. It will be fixed after this: #11132

Comment thread tests/python/contrib/test_hexagon/topi/test_depthwise_conv2d.py Outdated
)


class TestDepthwiseConv2D_More(BaseDepthwiseConv2D):
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can you rename this to TestDepthwiseConv2D?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes.

Comment thread tests/python/contrib/test_hexagon/topi/test_depthwise_conv2d.py
Copy link
Copy Markdown
Member

@mehrdadh mehrdadh left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @farshidsp!

@mehrdadh mehrdadh merged commit efbd721 into apache:main Apr 28, 2022
shtinsa pushed a commit to Deelvin/tvm that referenced this pull request May 17, 2022
* Add test for registered scheduales - depthwise_conv2d
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