Skip to content

Conversation

@cconvey
Copy link
Contributor

@cconvey cconvey commented Jul 21, 2022

  • Add utility functions to allow more human-readable pytest test IDs.
    Helpful when ID strings become too large for humans to easily read.

  • Update the test_avg_pool2d_slice.py unit test to use this mechanism.

cc @mehrdadh

@cconvey
Copy link
Contributor Author

cconvey commented Jul 21, 2022

CC: @Lunderberg

@cconvey
Copy link
Contributor Author

cconvey commented Jul 21, 2022

Example of the new ID formatting:

======================================================================== test session starts =========================================================================
platform linux -- Python 3.8.0, pytest-7.1.2, pluggy-1.0.0 -- /usr/bin/python3.8
cachedir: .pytest_cache
rootdir: /home/cconvey/r/cconvey/tvm/pytest-ids
collected 14 items                                                                                                                                                   

tests/python/contrib/test_hexagon/topi/test_avg_pool2d_slice.py::TestAvgPool2dSlice::test_avg_pool2d_slice[nhwc-8h2w32c2w-2d-out_shape[1,8,8,32]-kernel[3,3]-stride[1,1]-dil[1,1]-pad[0,0,0,0]-ceil:F-cnt_padded:T-out_layout:nhwc-8h2w32c2w-2d-float16] PASSED

@cconvey
Copy link
Contributor Author

cconvey commented Jul 22, 2022

@jverma-quic : FYI because you authored one of these files.

- Add utility functions to allow more human-readable pytest test IDs.
  Helpful when ID strings become too large for humans to easily read.

- Update the `test_avg_pool2d_slice.py` unit test to use this mechanism.
@kparzysz-quic kparzysz-quic merged commit 9863cf0 into apache:main Jul 22, 2022
@cconvey cconvey deleted the pytest-ids branch July 22, 2022 21:11
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
- Add utility functions to allow more human-readable pytest test IDs.
  Helpful when ID strings become too large for humans to easily read.

- Update the `test_avg_pool2d_slice.py` unit test to use this mechanism.
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
- Add utility functions to allow more human-readable pytest test IDs.
  Helpful when ID strings become too large for humans to easily read.

- Update the `test_avg_pool2d_slice.py` unit test to use this mechanism.
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