Dr-Noob / cpufetch

Simple yet fancy CPU architecture fetching tool
GNU General Public License v2.0
1.9k stars 103 forks source link

Add short versions of ASCII arts #283

Open hrw opened 2 months ago

hrw commented 2 months ago

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 commented 2 months ago
                                          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.

Dr-Noob commented 2 months ago

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.

hrw commented 2 months ago

I would rather add --no-logo argument.

No ANSI gfx skills

Dr-Noob commented 2 months ago

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 commented 2 months ago

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