Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong output, should be MiB instead of MB #7

Open
vovaf709 opened this issue Dec 19, 2022 · 5 comments
Open

Wrong output, should be MiB instead of MB #7

vovaf709 opened this issue Dec 19, 2022 · 5 comments

Comments

@vovaf709
Copy link

nvidia-smi shows MiB-s, but not MB-s, so reprs of corresponding objects should be changed

@KwangryeolPark
Copy link

MB means Mega Byte so MiB is MB.

@vovaf709
Copy link
Author

MiB mean mebibyte, see this table https://en.wikipedia.org/wiki/Byte#Multiple-byte_units

@KwangryeolPark
Copy link

KwangryeolPark commented Aug 22, 2023

@vovaf709 According to your reference, MiB and MB are same unit because the MB in nvidia-smi means Memory. See the table

@vovaf709
Copy link
Author

I am confused) nvidia-smi shows MiB-s as it is denoted in the command output

image

But nvsmi shows same numbers with MB-s
image

Here nvidia-smi-s 16376 MiB-s vs nvsmi-s 16376 MB-s. Isn't is wrong?

@KwangryeolPark
Copy link

I am confused) nvidia-smi shows MiB-s as it is denoted in the command output

image

But nvsmi shows same numbers with MB-s image

Here nvidia-smi-s 16376 MiB-s vs nvsmi-s 16376 MB-s. Isn't is wrong?

Theoretically, MiB and MB is different unit as you mentioned, but in general they work like same unit in computer scient as I know. However, it looks better to use MiB. So, it is better to fork this repository and fix MB to MiB at nvsmi/nvsmi.py line 61, 79, 228. After then, you can install the package with

pip install git+<your repository url>

for example

pip install git+https://github.com/xxxxxx/nvsmi.git

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants