Skip to content

Update model names to human-readable format for presentation (#447)#472

Merged
ctr-lelanchelianTT merged 2 commits intomainfrom
lelanchelian/fix_variant_name_underscores
Feb 12, 2026
Merged

Update model names to human-readable format for presentation (#447)#472
ctr-lelanchelianTT merged 2 commits intomainfrom
lelanchelian/fix_variant_name_underscores

Conversation

@ctr-lelanchelianTT
Copy link
Contributor

Ticket

closes tenstorrent/tt-xla#2970

Problem description

Currently, model architecture and variant name are more in the form of encoding rather than the actual name. We should update them to represent human-readable names used for model presentation purposes

What's changed

This PR updates them to use human-readable naming conventions. As part of this change, the model variant names have been corrected, and in some places where the ModelVariant class incorrectly used author names, those have been fixed as well. This aligns with the guidelines specified in the
README . I have tested the affected models individually, and the corresponding logs are attached for reference.

Copy link
Contributor

@nvukobratTT nvukobratTT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few small comments for changes I spot. Approving

@ctr-lelanchelianTT ctr-lelanchelianTT force-pushed the lelanchelian/fix_variant_name_underscores branch 4 times, most recently from 3279174 to 738428e Compare February 11, 2026 09:19
### Ticket
closes tenstorrent/tt-xla#2970

### Problem description
Currently, model architecture and variant name are more in the form of
encoding rather than the actual name. We should update them to represent
human-readable names used for model presentation purposes

### What's changed
This PR updates them to use human-readable naming conventions. As part
of this change, the model variant names have been corrected, and in some
places where the ModelVariant class incorrectly used author names, those
have been fixed as well. This aligns with the guidelines specified in
the
[README](https://github.com/tenstorrent/tt-forge-models#:~:text=Always%20use%20a%20dedicated%20ModelVariant%20enum%20class%20with%20a%20DEFAULT_VARIANT%20class%20attribute)
. I have tested the affected models individually, and the corresponding
logs are attached for reference.
@ctr-lelanchelianTT ctr-lelanchelianTT merged commit 18d662b into main Feb 12, 2026
2 checks passed
@ctr-lelanchelianTT ctr-lelanchelianTT deleted the lelanchelian/fix_variant_name_underscores branch February 12, 2026 05:03
ctr-pmuruganTT added a commit to tenstorrent/tt-xla that referenced this pull request Feb 16, 2026
### Ticket
Ref : [Nightly
failure](https://github.com/tenstorrent/tt-xla/actions/runs/21969404262/job/63467192440#step:17:223)

### Problem description
Whisper model is failing with an error in nightly because of a recent
naming convention
[update](tenstorrent/tt-forge-models#472)

error is `AttributeError: 'Seq2SeqModelOutput' object has no attribute
'logits'`

### What's changed
* updated the latest variant names as per new convention
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

Successfully merging this pull request may close these issues.

Update model naming

2 participants