Re: [yew-devel] [PATCH proxmox-yew-comp] fix: use IEC standared for showing the HD space

["Shan Shaji" <s.shaji@proxmox.com>]

On Mon Mar 9, 2026 at 10:22 PM CET, Thomas Lamprecht wrote:
> Am 19.12.25 um 18:21 schrieb Shan Shaji:
>> The HD space was being showing in SI unit, However all other metrics
>> shown in the remote overview section were in IEC unit. Fixed it by
>> updating it to use the IEC unit.
>
> But we do favor base-10 units for storage sizes and IEC for memory (and
> bits for network traffic), so this would seem alright as is to me, or?

Thank you Thomas. I was not aware about that. However,  when i checked PVE,
inside the summary section the HD space is shown in IEC unit. For PBS, the
HD space under dashboard is in base-10. As for PDM, under the remote dasboard
the "Host Storage used" is shown in IEC unit. I believe it's a bit inconsistent.
I am not sure if that's expected. Shall i make it consistent across the products as
you mentioned?

>> 
>> Signed-off-by: Shan Shaji <s.shaji@proxmox.com>
>> ---
>>  src/node_info.rs | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/src/node_info.rs b/src/node_info.rs
>> index 0f8fcb5..ca656d0 100644
>> --- a/src/node_info.rs
>> +++ b/src/node_info.rs
>> @@ -204,8 +204,8 @@ pub fn node_info(data: Option<NodeStatus>) -> Container {
>>                  .status(format!(
>>                      "{:.2}% ({} of {})",
>>                      fraction * 100.0,
>> -                    HumanByte::new_decimal(root_used as f64),
>> -                    HumanByte::new_decimal(root_total as f64),
>> +                    HumanByte::from(root_used),
>> +                    HumanByte::from(root_total),
>>                  ))
>>          })
>>          .with_child({