Skip to content

Conversation

@quic-sanirudh
Copy link
Contributor

Second set of hexagon tests modified to be pylint compliant as part of #11414 tracking issue. The files supported in this patch are:

  • tests/python/contrib/test_hexagon/test_autotvm.py
  • tests/python/contrib/test_hexagon/test_cache_read_write.py
  • tests/python/contrib/test_hexagon/test_launcher.py
  • tests/python/contrib/test_hexagon/test_maxpool2d_blocked.py
  • tests/python/contrib/test_hexagon/test_models.py
  • tests/python/contrib/test_hexagon/test_run_unit_tests.py
  • tests/python/contrib/test_hexagon/test_thread_pool.py
  • tests/python/contrib/test_hexagon/test_usmp.py

@quic-sanirudh quic-sanirudh force-pushed the fix_linter_hexagon_tests branch from 88f2d5d to 9cb40d4 Compare July 27, 2022 17:20
@quic-sanirudh
Copy link
Contributor Author

@Mousius Could you take a look at this second set of hexagon tests that are added to be pylint compliant? Thanks.

Copy link
Member

@junrushao junrushao left a comment

Choose a reason for hiding this comment

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

LGTM

@junrushao junrushao merged commit 4a6d655 into apache:main Aug 2, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
)

Second set of **hexagon tests** modified to be pylint compliant as part of apache#11414 tracking issue. The files supported in this patch are:

* [X] tests/python/contrib/test_hexagon/test_autotvm.py
* [X] tests/python/contrib/test_hexagon/test_cache_read_write.py
* [X] tests/python/contrib/test_hexagon/test_launcher.py
* [X] tests/python/contrib/test_hexagon/test_maxpool2d_blocked.py
* [X] tests/python/contrib/test_hexagon/test_models.py
* [X] tests/python/contrib/test_hexagon/test_run_unit_tests.py
* [X] tests/python/contrib/test_hexagon/test_thread_pool.py
* [X] tests/python/contrib/test_hexagon/test_usmp.py
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