Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: correct theme memory display template for byte/gigabyte conversion #4557

Merged

Conversation

ChrisNSki
Copy link
Contributor

@ChrisNSki ChrisNSki commented Dec 19, 2023

Updated conversion factor from 1000000000 to 1073741824. This change ensures memory usage is correctly displayed as a gigabyte (2^30 bytes).

Prerequisites

  • I have read and understood the contributing guide.
  • The commit message follows the conventional commits guidelines.
  • Tests for the changes have been added (for bug fixes / features).
  • Docs have been added/updated (for bug fixes / features).

Description

This pull request fixes the byte-to-gigabyte conversion in 7 oh-my-posh theme templates. The original templates used 1,000,000,000 as the divisor. This update changes the divisor to the value of 1,073,741,824 (2^30 bytes), this should resolve an issues with accurate gigabyte representation in memory usage displays on those themes.

…rsion

Updated conversion factor from 1000000000 to 1073741824. This change ensures memory usage is correctly displayed as a gigabyte (2^30 bytes).
@JanDeDobbeleer JanDeDobbeleer merged commit e976ce8 into JanDeDobbeleer:main Dec 19, 2023
15 checks passed
@JanDeDobbeleer
Copy link
Owner

@all-contributors please add @ChrisNSki for design

This is pure magic 🪄! @holopin-bot @ChrisNSki wizard

Copy link

holopin-bot bot commented Dec 19, 2023

Congratulations @ChrisNSki, you just earned a badge! Here it is: https://holopin.io/claim/clqcmprs013180fjmshz0v9av

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply # with GitHub, which will do the trick!

Copy link
Contributor

@JanDeDobbeleer

I've put up a pull request to add @ChrisNSki! 🎉

@ChrisNSki ChrisNSki deleted the fix/template-memory-conversion branch December 19, 2023 17:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants