mlpstorage arguments have confusing names that conflict with IEC standards

[jeking3]

According to the code, CLIENT_HOST_MEMORY_IN_GB is actually in GiB:
https://github.com/search?q=repo%3Amlcommons%2Fstorage%20CLIENT_HOST_MEMORY_IN_GB&type=code

    elif args.client_host_memory_in_gb and args.num_client_hosts:
        # If host memory per client and num clients is provided, we use these values instead of the calculated memory
        per_host_memory_in_bytes = args.client_host_memory_in_gb * 1024 * 1024 * 1024

The command takes --client-host-memory-in-gb as an argument, not --client-host-memory-in-gib. These distinctions are important. The README.md and command help should be updated to make it clear this is in GiB units. Refer to https://physics.nist.gov/cuu/Units/binary.html for more details.

[jeking3]

also:

storage/mlpstorage/benchmarks/dlio.py

Line 226 in 988ebc0

self.logger.result(f'Total disk space required for training: {total_disk_bytes / 1024**3:.2f} GB')