tenstorrent / pytorch2.0_ttnn

⭐️ TTNN Compiler for PyTorch 2.0 ⭐️ It enables running PyTorch2.0 models on Tenstorrent hardware
https://tenstorrent.github.io/tt-metal/latest/ttnn/
25 stars 5 forks source link

Model Pose Landmark. #141

Closed cthsieh closed 1 month ago

cthsieh commented 2 months ago

Ticket

None

Problem description

Add model Pose Landmark to trace.

This test depends on fixture manage_depedencies which was developed in PR 137. So, review PR 137 first, and then I continue to finish this PR in case of any change according to PR 137.

What's changed

cthsieh commented 1 month ago

@ayerofieiev-tt
We need to install Git LFS in the CI for this PR to pass.

Currently, the PR is encountering an error related to opening a file, which is specifically an LFS file.

E     RuntimeError: Unable to open file at /home/ubuntu/actions-runner/_work/pytorch2.0_ttnn/pyto\
rch2.0_ttnn/pose_landmarker_lite.task

This PR contains two LFS files.

I also expanded README.md to describe the use of Git LFS.

ayerofieiev-tt commented 1 month ago

Installed. Running on CI to check

cthsieh commented 1 month ago

I found that the missing file problem involves several causes:

  1. Not installed Git LFS. ✅ Solved as you said.
  2. Wrongly set the file path in the test. ✅ Solved by the commits I just sent.
  3. Not ran git lfs pull to actually get the content of LFS files. ❓

The CI still failed and I suspect it's the 3rd cause.

https://github.com/tenstorrent/pytorch2.0_ttnn/actions/runs/10858105217/job/30135655187 The error is about unrecognized file format, which I guess, is now the LFS pointer content before downloaded.

E     RuntimeError: Unable to open zip archive.

To confirm my guess, could you please check the CI server to see the file size of the two LFS files: tests/models/pose_landmark/pose_landmarker_lite.task and tests/models/pose_landmark/test_image.jpg? The file size should not be small about a few bytes.