Memory bar display fails if anything other than 'mib' is used for memory_unit

[claytonjroberts]

Description

When using a bar for memory info (memory_display="bar"), I must have the following in the config file:

memory_unit="mib"

If I have something like the following in the config:

memory_unit="gib"

I get this error:

/usr/local/bin/neofetch: line 4583: 18.50 * bar_length / 32.00: syntax error: invalid arithmetic operator (error token is ".50 * bar_length / 32.00")

Note: I'm running via just neofetch -v

• Does this issue still occur in the master branch? (Required if issue)

Neofetch version

7.1.0

Screenshot

Config file

https://gist.github.com/ClaymasterJ/00b8702da639c0ccc2913e7802dc8a56
(^ Working config)

Verbose log

https://pastebin.com/nyfwNbLP

[deergit]

I fixed this by changing lines 2655 and 2656:

2655: - mem_used=$(awk '{printf "%.2f", $1 / $2}' <<< "$mem_used 1024")
2656: - mem_total=$(awk '{printf "%.2f", $1 / $2}' <<< "$mem_total 1024")

2655: + mem_used=$(awk '{printf "%.0f", $1 / $2}' <<< "$mem_used 1024")
2656: + mem_total=$(awk '{printf "%.0f", $1 / $2}' <<< "$mem_total 1024")

the math fails because bash doesn't handle floats, however if we change the floats to have no decimals it can work again

[hykilpikonna]

This issue was fixed in #2225