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

Binary prefixes #33

Open
mueslo opened this issue Dec 4, 2016 · 7 comments
Open

Binary prefixes #33

mueslo opened this issue Dec 4, 2016 · 7 comments

Comments

@mueslo
Copy link

mueslo commented Dec 4, 2016

When visualising information, it would be quite useful to have the option of not just SI prefixes but also IEC binary prefixes, i.e. 1024 == Ki, 1048675 == Mi, etc.

@mueslo mueslo changed the title Binary prefix Binary prefixes Dec 4, 2016
@mbostock
Copy link
Member

mbostock commented Dec 4, 2016

Sounds like a reasonable suggestion. I have no immediate plans to implement this but would be willing to review a pull request that implements it.

nicksrandall added a commit to nicksrandall/d3-format that referenced this issue Feb 28, 2017
This could serve as an escape hatch for somebody who wanted to support custom SI prefixes. This could be used to solve d3#33 (binary prefixes) or it could be used to support something custom like American english numbers (1,000,000,000 => 1B [billion] instead of 1G [giga]).
runarberg pushed a commit to runarberg/d3-format that referenced this issue May 6, 2019
runarberg pushed a commit to runarberg/d3-format that referenced this issue Jul 25, 2019
runarberg pushed a commit to runarberg/d3-format that referenced this issue Sep 2, 2019
runarberg pushed a commit to runarberg/d3-format that referenced this issue Mar 28, 2020
Fil pushed a commit that referenced this issue Jul 7, 2020
runarberg pushed a commit to runarberg/d3-format that referenced this issue Nov 11, 2020
runarberg pushed a commit to runarberg/d3-format that referenced this issue Nov 11, 2020
@Fil
Copy link
Member

Fil commented Aug 23, 2023

#70 is a good option (needs to be rebased on main to be reopened).

runarberg pushed a commit to runarberg/d3-format that referenced this issue Aug 23, 2023
runarberg pushed a commit to runarberg/d3-format that referenced this issue Aug 23, 2023
@runarberg
Copy link

@Fil I rebased #70 on top of main and ran a couple of macros on the tests to keep them up to date. Let me know if there is anything else you need 😉

@grotlue
Copy link

grotlue commented Sep 29, 2023

Hello again👋
Is there any chance that #70 is being merged?

@Fil
Copy link
Member

Fil commented Sep 29, 2023

#70 lgtm

@grotlue
Copy link

grotlue commented Oct 13, 2023

Great! Is it possible to merge it to d3-format main?

@ldayananda
Copy link

Since it seems based on this comment that the IEC prefixes got stalled due to the currency changes -- would it be possible to extract the IEC changes to the main project?

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

Successfully merging a pull request may close this issue.

6 participants