Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Watcher env var to dump logical/virtual/physical coords in the log #17083

Open
tt-dma opened this issue Jan 24, 2025 · 0 comments
Open

Watcher env var to dump logical/virtual/physical coords in the log #17083

tt-dma opened this issue Jan 24, 2025 · 0 comments
Assignees

Comments

@tt-dma
Copy link
Contributor

tt-dma commented Jan 24, 2025

Watcher log currently shows logical and virtual coordinates.

The thinking is that physical coordinates are needed for power users (runtime team) and that we can expose them in the watcher log if an env var is set (say, TT_METAL_WATCHER_SHOW_PHYSICAL_COORDS).

We wanted to avoid potentially showing an example of extracting and using physical coords, so plan is to add a cluster api to get physical coord from logical, which includes a (disable-able by non-default argument) warning to not use physical coords.

FYI @pgkeller @tt-asaigal

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

No branches or pull requests

1 participant