Skip to content

Navigation Menu

Explore
By company size
By use case
By industry
View all solutions
Topics
- AI
- DevOps
- Security
- Software Development
- View all
Explore
- GitHub Sponsors
  Fund open source developers
- The ReadME Project
  GitHub community articles
Repositories
- Enterprise platform
  AI-powered developer platform
Available add-ons
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

tenstorrent / tt-metal Public

Notifications You must be signed in to change notification settings
Fork 90
Star 587

Code
Issues 1.9k
Pull requests 316
Discussions
Actions
Projects
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Discussions
Actions
Projects
Security
Insights

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.

Sign up for GitHub

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

Jump to bottom

[UMD] Switch to .get_cores api in soc_descriptor #16598

Draft

broskoTT wants to merge 1 commit into main

base: main

Choose a base branch

Loading

Loading

from brosko/get_cores_api

Draft

[UMD] Switch to .get_cores api in soc_descriptor #16598

broskoTT wants to merge 1 commit into main from brosko/get_cores_api

Conversation 0 Commits 1 Checks 10 Files changed

Conversation

Copy link

Contributor

broskoTT commented Jan 10, 2025 •

edited

Loading

Ticket

Related to

Problem description

Switch to new API which provides CoreCoord access.
This is all due to harvesting being moved out of soc_descriptor into its coordinate manager.

What's changed

All .workers accesses are changed to .get_cores call.
Similar for .eth and others

Checklist

All runs on brosko/get_cores_api after just UMD bump :

All post-commit tests : https://github.com/tenstorrent/tt-metal/actions/runs/12708022091
Blackhole post-commit tests : https://github.com/tenstorrent/tt-metal/actions/runs/12708023828
(Single-card) Model perf tests : https://github.com/tenstorrent/tt-metal/actions/runs/12708025849
(Single-card) Device perf regressions : https://github.com/tenstorrent/tt-metal/actions/runs/12708027779
(T3K) T3000 unit tests : https://github.com/tenstorrent/tt-metal/actions/runs/12708029487
(T3K) T3000 demo tests : https://github.com/tenstorrent/tt-metal/actions/runs/12708030811
(TG) TG unit tests : https://github.com/tenstorrent/tt-metal/actions/runs/12708032478
(TG) TG demo tests : https://github.com/tenstorrent/tt-metal/actions/runs/12708034126
(TGG) TGG unit tests : https://github.com/tenstorrent/tt-metal/actions/runs/12708036061
(TGG) TGG demo tests : https://github.com/tenstorrent/tt-metal/actions/runs/12708037559

Sorry, something went wrong.

All reactions


          bump umd

f1c2e3e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

abhullar-tt Awaiting requested review from abhullar-tt abhullar-tt will be requested when the pull request is marked ready for review abhullar-tt is a code owner

pgkeller Awaiting requested review from pgkeller pgkeller will be requested when the pull request is marked ready for review pgkeller is a code owner

aliuTT Awaiting requested review from aliuTT aliuTT will be requested when the pull request is marked ready for review aliuTT is a code owner

tt-aho Awaiting requested review from tt-aho tt-aho will be requested when the pull request is marked ready for review tt-aho is a code owner

tt-dma Awaiting requested review from tt-dma tt-dma will be requested when the pull request is marked ready for review tt-dma is a code owner

tt-asaigal Awaiting requested review from tt-asaigal tt-asaigal will be requested when the pull request is marked ready for review tt-asaigal is a code owner

ubcheema Awaiting requested review from ubcheema ubcheema will be requested when the pull request is marked ready for review ubcheema is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.

Footer

© 2025 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Docs
Contact

You can’t perform that action at this time.