tenstorrent / tt-smi

Tenstorrent console based hardware information program
Apache License 2.0
21 stars 6 forks source link

Allow specifying target filename for JSON config output #6

Closed tt-rkim closed 6 months ago

tt-rkim commented 7 months ago

Right now, when using -s, it outputs a JSON with a pre-determined filename, appending a date.

This is not super great for automation. Could we have the ability to set a snapshot name just like the old SMI?

sbansalTT commented 7 months ago

Hey Raymond, we have it already added that feature in, you have to use the -f flag to set the filename of the snapshot, exactly like old SMI. Link to instructions - https://github.com/tenstorrent/tt-smi?tab=readme-ov-file#snapshots

tt-rkim commented 7 months ago

Hey @sbansalTT - I mean with the -g option, not the -s option.

For example, here, it would be nice if the first one generated log.log instead of a pre-determined JSON filename.

Screen Shot 2024-02-21 at 11 35 42 AM
sbansalTT commented 7 months ago

Oh gotcha - yes, I will add that into the next release

sbansalTT commented 6 months ago

This has been fixed as of release 2.1.0 you should now be able to specify the file, or use the default with -g when generating the reset config