Skip to content

[TOPI][Hexagon] Add test and schedule for uint8 resize2d#12559

Merged
kparzysz-quic merged 9 commits into
apache:mainfrom
trahman-quic:trahman-quic/resize2d_uint8
Sep 9, 2022
Merged

[TOPI][Hexagon] Add test and schedule for uint8 resize2d#12559
kparzysz-quic merged 9 commits into
apache:mainfrom
trahman-quic:trahman-quic/resize2d_uint8

Conversation

@trahman-quic
Copy link
Copy Markdown
Contributor

@trahman-quic trahman-quic commented Aug 23, 2022

This PR adds schedule for float16 and uint8 and int8 layout. It also adds testing for uint8 layout. Additionally, it removes testing for cubic method since SoW models don't test for it and the current implementation of resize2d in topi does not have appropriate support for it.

cc @mehrdadh

@github-actions github-actions Bot requested a review from mehrdadh August 23, 2022 17:11
@trahman-quic
Copy link
Copy Markdown
Contributor Author

The details for the changes in this commit for this PR can be referred to here

@trahman-quic trahman-quic force-pushed the trahman-quic/resize2d_uint8 branch from 3bf4eae to 584143d Compare August 23, 2022 18:05
@mehrdadh
Copy link
Copy Markdown
Member

mehrdadh commented Sep 1, 2022

@trahman-quic please update the PR description. The link that you provided is not open to public

@trahman-quic trahman-quic force-pushed the trahman-quic/resize2d_uint8 branch from 58e3c5c to af7e7f9 Compare September 2, 2022 20:05
@trahman-quic trahman-quic force-pushed the trahman-quic/resize2d_uint8 branch from af7e7f9 to 158dc68 Compare September 2, 2022 20:09
Comment thread tests/python/contrib/test_hexagon/topi/test_resize2d.py Outdated
@trahman-quic trahman-quic force-pushed the trahman-quic/resize2d_uint8 branch from 747931f to 1276774 Compare September 8, 2022 17:54
@kparzysz-quic kparzysz-quic merged commit 4c05656 into apache:main Sep 9, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
* [TOPI][Hexagon] Add test and schedule for uint8 resize2d

* Fix correctness issue

* Reformat

* Remove cubic from testing

* Remove unnecessary else
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.

4 participants