improve size column readability in archives tab

[hariseldon78]

I find hard to read the sizes of the archives at a glance when they are all mixed units: usually the longer numbers represent smaller sizes, making it difficult to find the biggest archives. So I made a basic algorithm to select a good unit that can adequately represent all sizes, that makes it much easier to read. Also I made the column align to the right, which further improves readability.

Related Issue

#1595

Motivation and Context

Readability

How Has This Been Tested?

Tested on my archive list, which sizes goes from about 100 MB to 150 GB

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING guide.
• I have added tests to cover my changes.
• All new and existing tests passed.

I provide my contribution under the terms of the license of this repository and I affirm the Developer Certificate of Origin.

[real-yfprojects]

Please add signature type hints and docstrings for pretty_bytes and find_best_size_formatting.

[real-yfprojects]

Pre-commit will format your code (including sorting imports) automatically.

[hariseldon78]

unfortunately pre-commit is not working on my system, this is the error I receive:
pre-commit.log
the fixes I found talks about upgrading the isort package, but the version is not pinned in the requirements file, so it should already be at the highest version compatible with the other packages...

[Edit] ok this problem is related to #1601, so i've been able to install and run it by doing the pre-commit autoupdate

[real-yfprojects]

[Edit] ok this problem is related to #1601, so i've been able to install and run it by doing the pre-commit autoupdate

Rebasing your branch onto master will also do the job.

It won't since the corresponding PR is not merged yet.

[real-yfprojects]

Great, you added test cases and cleverly improved my code!

[real-yfprojects]

Can you add a test for a very large size like $10^{30}$?

[hariseldon78]

Can you add a test for a very large size like 1030?

it was failing actually :)

[real-yfprojects]

Only some minor typing problems which make them even more minor.

[m3nu]

Thanks all! Give me until this evening to run this locally.

[m3nu]

Works as advertised. Interesting addition. Thanks for your work and patience on this!