cpufetch icon indicating copy to clipboard operation
cpufetch copied to clipboard

Add short versions of ASCII arts

Open hrw opened this issue 1 year ago • 6 comments

There is nearly no space for information:

[marcin@sbc ~]$ /tmp/cpufetch

                                                          SoC:          Rockchip
                                                          Technology:   8nm
                                                          CPU 1:
                                              ##            uArch:      Cortex-A
   ######                ##            ##                   Max Freq:   1.800 GH
  ##.  ### ##### #####  ## .## #####  ######. ##   #####    Cores:      4 cores
 #######. ##. # #.     #####  ##.    ###  ###  #     .##    Features:   NEON,SHA
##.  ###.  ####. #### ###  .## #### ###  ###   #.#####    CPU 2:
                                                ##          uArch:      Cortex-A
                                                            Max Freq:   2.400 GH
                                                            Cores:      4 cores
                                                            Features:   NEON,SHA
                                                          Peak Perf.:   211.20 G

hrw avatar Sep 09 '24 09:09 hrw

                                          SoC:          Unknown
   #####  ##   # #####  ## ####  ######   Technology:   Unknown
 ###    ####   ###      ####  ###   ###   uArch:        Neoverse N2
###       ##   ###      ###    ##    ###  Max Freq:     ~960 MHz
 ###    ####   ###      ###    ##    ###  Cores:        4 cores
  ######  ##   ###      ###    ##    ###  Features:     NEON,SVE,SVE2,SHA1,SHA2,
                                          Peak Perf.:   61.44 GFLOP/s

'Features' field is cut.

hrw avatar Sep 09 '24 09:09 hrw

Rochckip logo is too big, I agree. Usually there is a long version of the logos and a short one (you show here the long Rockchip one and the ARM short one), but Rockchip has no logo version at the moment.

ARM short logo (the one you show) is not too wide in my opinion. Some space for the logo is needed.

If you believe you can design shorter versions without making them look ugly please feel free to open a PR, thanks.

Dr-Noob avatar Sep 10 '24 07:09 Dr-Noob

I would rather add --no-logo argument.

No ANSI gfx skills

hrw avatar Sep 10 '24 08:09 hrw

I would have alias cpufetch --no-logo=lscpu (or any other tool).

One of the keys of cpufetch is the logo. Yes, in small terminals this can be an issue, but in general it fits nicely, except for vendors which do not have short logos like rockchip. But that can be solved with shorter logos, which is not a priority right now, but eventually they will be added as well.

Dr-Noob avatar Sep 11 '24 17:09 Dr-Noob

I'll have this issue as a remainder to work on this one day :+1:

Dr-Noob avatar Sep 11 '24 17:09 Dr-Noob

Same problem. Termux, and :

--logo-short

                                                           SoC:
                                                           Technology:
                                                           CPU 1:
                                                             uArch:
    ##  ## ######  ######  #     ### @@@@@@ @@@@@@ @@  @@    Max Freq:
   ### ### #       #    #  #    ####   @@   @      @@ @@     Cores:
  ######## # ###   #    #  #   ## ##   @@   @ @@@  @@@@      Features:
 ## ### ## #       #    #  #  ##  ##   @@   @      @@ @@   CPU 2:
##  ##  ## ######  #####   # ##   ##   @@   @@@@@@ @@  @@    uArch:
                                                             Max Freq:
                                                             Cores:
                                                             Features:
                                                           Peak Perf.:

root@localhost:~/downloads_Termux/cpufetch#

Manamama avatar Dec 12 '24 04:12 Manamama