Fabric icon indicating copy to clipboard operation
Fabric copied to clipboard

Added current model indicator with an * when --listmodels is called

Open mchoccie opened this issue 1 year ago • 2 comments

What this Pull Request (PR) does

Trying to add a small and simple feature for my first contribution. Whenever 'fabric --listmodels' is called, the output will denote the currently used model with an asterisk after it. Hopefully this will give users a better idea of the current model they've set Fabric to utilize.

Related issues

closes #446

Screenshots

image

mchoccie avatar Jul 03 '24 01:07 mchoccie

Just encountered the same thing and would love having it in. Would maybe write a model print helper function, to not have so many if repetitions and only one place to change the pattern of what is the current model and what not maybe would also have instead of a * something like (current)

auryn31 avatar Jul 03 '24 11:07 auryn31

Hey @auryn31 , I implemented the changes you mentioned. What do you think?

mchoccie avatar Jul 04 '24 09:07 mchoccie

This is great, but we're moving to Go. Please resubmit if this isn't fixed after the migration. Thank you!

danielmiessler avatar Aug 16 '24 19:08 danielmiessler