search

tenstorrent / tt-metal

:metal: TT-NN operator library, and TT-Metalium low level kernel programming model.
Apache License 2.0
303 stars 25 forks source link

Conv layer (low PCC) in yolov4 Head module #8640

Open saichandax opened 1 month ago

saichandax commented 1 month ago

Reproducible script for the Conv failing case is at the branch: https://github.com/tenstorrent/tt-metal/compare/main...ankit/head_c11_unittest

Please checkout to the branch and execute the unit test file.

git checkout ankit/head_c11_unittest
pytest tests/ttnn/unit_tests/operations/test_conv_head_c11_yolov4.py

We are currently getting Low PCC (0.6444111739627862).

Expected behavior: To achieve PCC of close to 0.99

tt-rkim commented 1 week ago

@mbahnasTT Can you explain why this is P0 now?

mbahnasTT commented 1 week ago

We are going to sync more with Jasmina and Sid on pumping the priority of some customer models, so this issue is among the ones that will enable the high priority customer models.

tt-rkim commented 1 week ago

OK no problem. Please update this soon with an explanation when you have it. Thank you.

punithsekar commented 2 days ago

Note: The unit test branch is up to date https://github.com/tenstorrent/tt-metal/compare/main...ankit/head_c11_unittest.