-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
Add integer formatting code for fixed-width signed arithmetic (2's complement) #74756
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
Comments
The current 'b' formatting code in inconvenient for emulating, demonstrating, and teaching two's complement arithmetic. The problem is that negative inputs are always formatted with a minus sign. I propose that some formatting code be provided for fixed-width display where the leading bit is a sign bit. For example, if code were a capital 'B' and the total width were 8-bits: >>> x = -12
>>> format(12, '08B')
'11110100' Currently, to achieve the same effect, one of the following is used: >>> format(x if x >= 0 else x + 2**8, '08b')
'11110100' or >>> format(x & (2**8 - 1), '08b')
'11110100' For values outside the valid range, perhaps a ValueError could be raised: >>> format(-200, '08B')
Traceback (most recent call last):
...
ValueError: Expected value in range -128 <= x < 128, not -200 I'm not sure what the right code should be. The idea of capital 'B' is attractive, but we already have a different relationship between 'X' and 'x'. There could also be a modifier symbol such as '!' in '!8b'. |
-1 from me. Using |
Concur with Mark. Similar issues already were raised multiple times on mailing lists and the conclusion is that explicit wrapping integers to specific range is better. Different behavior for integers out of range is needed in different applications. |
This is a recurring need in my teaching of Python to hardware engineers. The whole point of having a binary format code is to show which bits are set. For negative values, that need is not being served by the current option. And it makes it awkward when modeling the effects of bitwise operations on signed numbers. I can understand a -1 if you think this is fundamentally broken, but if you're just saying that you've never personally needed this or casually dismissing the awkwardness of the usual workarounds, then it seems like a meaningless vote that blows-off my proposal which is based on real-world classroom experiences. The idea is that modifier (such as "!" would require a width argument (the signed representation only make sense in fixed width concepts as every assembly language programmer knows). Its presence would also imply the "0". Serhiy, I believe you've either read different maillist posts than I have or that you're misinterpreting this as a proposal for a fixed width integer type with automatic wrap-around and truncation (full emulation of a register). This proposal is for display formatting only. IMO, it remedies a deficiency where the current option is more for our convenience (not wanting to deal with the width) rather than for the user's convenience where the user wants to see which bits are set rather than seeing a minus sign (which is both obvious and useless). Also, when a senior coredev presents a proposal, I expect that it will be greeted with a little more open mindedness and not instantly shot down as if I have no idea what I'm talking about. |
I would use the precision for this. Truncating the number of digits in integer representation is similar to truncating the number of characters for strings. But actually I don't think we should add such feature in the core. It has very limited application. This can be implemented as a third-part library. Actually I think the stdlib needs something like bitarray and/or bitset. The specially purposed types that allow testing and modifying separate bits and bit ranges. |
To expand on what Mark said: If the proposed format code implicitly >>> format(12 % 2**8, '08b')
'00001100'
>>> format(-12 % 2**8, '08b')
'11110100' ..., would students not expect that these two operations have the same >>> 0b11110100 | 0b00001100
252
>>>
>>> -12 | 12
-4 |
The format code space is already crowded, and it isn't immediately obvious that '08B' will format the 2's complement representation of the number, as opposed to '08b' which prepends a sign bit. Not to mention that, as Raymond remarked, we cannot use the same convention for hex formatting where 'X' already has a different meaning. Instead, how about adding an optional argument to bin()? And then we can also do the same for hex(). By the way, I do find annoying the default behaviour of prepending a sign bit on hexadecimal and binary displays (but I never use binary displays, so my annoyance is really with hexadecimal displays). |
I can't see the drawback of being able to format two's complement easily, isn't it more common to represent negative binary numbers in two's complement form? I find the - sign a little odd myself. I agree with the idea of using "!" with the precision to represent width. |
What's status of this issue? It seems, this idea got no wide support from core developers. Maybe it would be better to discuss that on https://discuss.python.org/c/ideas/6 ? |
I didn't expect "wide support". The group of people who work with binary representations of number is a small group. People who have never needed a programmer's calculator are unlikely to even recognize that the current 'b' format code is almost never useful for negative numbers. When a user has a real world need to see the a binary representation of a negative number, 99% of the time they want to see the two's complement. Pretty much the only reason to display a negative number in binary is to see how it is stored. FWIW for small features, our main criterion is usefulness. "Wide support" is too high of a bar; generally "some support" and no "fierce opposition" is sufficient. Also, it tends to be easier if the door has already been opened and we're fixing the utility of a feature. In this case, we already have a 'b' format code and a This particular feature is not often needed, but when you do need it, it is really inconvenient to not have it. |
I'm not sure what the best way to implement this in the format spec is, but me and my HP-16C (and now my Swissmicros DM16C) would definitely find this useful. |
That's exactly the case, sorry for being imprecise. So far, things looks more like (2).
Maybe (and same for "x" code). Though, e.g. But probably this ship is sailed. Or not?
Maybe we can fix this flaw instead? (BTW, now C has 'B' format type.) As Serhiy suggested above, I would use precision field to specify fixed-width. Recently, there was a thread, suggested to add precision support for new-style string formatting of integer types: https://discuss.python.org/t/80760/. Then, for "b", "x" and "X" formatting types - we can show two's complements either with a specified precision or "big enough" for a given negative value. Backward-compatible version of above - show two's complements iff precision is specified.
Most strong argument against from this issue thread was (for me): "The format code space is already crowded". I don't expect this feature will be discoverable for people (unless being a default for b/x formatting types). BTW, just recently while debugging I quickly ended up with a code snippet like in Mark's reply. This was easy and obvious. |
```pycon >>> f"{-12:.8b}" '11110100' >>> f"{200:.8b}" Traceback (most recent call last): File "<python-input-5>", line 1, in <module> f"{200:.8b}" ^^^^^^^^^ OverflowError: Expected integer in range [-2**7, 2**7) >>> f"{123:.8d}" '00000123' >>> f"{-12:.8d}" '-00000012' ```
pr is available to review: #131926 Twos complement used here for non-decimal integer formatting types iff precision was specified. |
PEP in progress for Precision and Modulo-Precision format specifiers. I'll keep posting. @skirpichev I'd hold off on the PR for now. |
Ok, it seems PEP going to be sponsored: python/peps#4365. Thus, I'm closing my pr. @jb2170, please either revert part, related to this issue or ensure Raymond is ok with your design. He was asking for formatting in two's complement representation, not for a shortcut |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
Linked PRs
The text was updated successfully, but these errors were encountered: