Skip to content

add mtt show - #1204

Open
sofiia-chorna wants to merge 2 commits into
metatensor:mainfrom
sofiia-chorna:mtt-show
Open

add mtt show#1204
sofiia-chorna wants to merge 2 commits into
metatensor:mainfrom
sofiia-chorna:mtt-show

Conversation

@sofiia-chorna

@sofiia-chorna sofiia-chorna commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Please go the the Preview tab and select the appropriate PR template:


📚 Documentation preview 📚: https://metatrain--1204.org.readthedocs.build/en/1204/

@sofiia-chorna
sofiia-chorna requested review from Luthaf and pfebrer July 15, 2026 17:54
@pfebrer

pfebrer commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

I think this is a good idea. However since it is adding another command we probably need to discuss it in mldevel. Maybe it helps to prepare:

  • What is the specific use case in which you have needed this?
  • Can you think of future functionalities that would also be integrated in this command?

For example, people requested also a command for editing models. Should these things go into the same command or would it be a separate one.

@sofiia-chorna

Copy link
Copy Markdown
Collaborator Author

thanks a lot for the quick feedback and your questions! happy to bring this to mldevel

the use case mentioned on the previous mldevel was finetuning (checking the heads of a checkpoint to know exact target name). from my side, oh if i take the checkpoint of a model i was not using before, it is nice to quickly check generally what is in there

for the extensions, we can show the hypers as well (e.g. with an additional flag). i would think more about other ideas & let's see if the meeting will bring anything

for editing, i lean to keep show strictly read-only. i'm curious what exactly people want to edit ^^ if it's renaming heads, oh might be tricky to do it general way as it would need to touch more of architecture specific state dict (heads / last layers / scaler etc)

@ceriottm

Copy link
Copy Markdown
Contributor

My 2 cents: (1) I agree this should be RO (but that it's good to discuss the command line options so that it matches those in a potential mtt edit command; (2) I think mtt info would be more obviously descriptive of what the command does.

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.

3 participants