test_sweep_conv_tt - the test relies on TT_METAL_HOME variable to be set

tenstorrent / tt-metal

:metal: TT-NN operator library, and TT-Metalium low level kernel programming model.

Apache License 2.0

396 stars 49 forks source link

test_sweep_conv_tt - the test relies on TT_METAL_HOME variable to be set #11257

Closed dimitri-tenstorrent closed 1 week ago

dimitri-tenstorrent commented 1 month ago

Test location: test_sweep_conv_tt

Code location: https://github.com/tenstorrent/tt-metal/blob/614b053a2ded8ab62c3e35c3bf68751b4962b1d7/tests/tt_eager/python_api_testing/conv/pytorch_conv_tb.py#L60

Convert the test case to lookup conv_sweep_params.yaml relative to the test location as opposed to TT_METAL_HOME location. This is particularly a problem with the wheel installation.

mywoodstock commented 1 month ago

Hey @tt-aho, are you familiar with this script?

tt-aho commented 1 month ago

I'm not, this was something @tt-nshanker added/was managing I think

dimitri-tenstorrent commented 2 weeks ago

Fixed in the PR #11132