Mark Dickinson added the comment:
New changeset c689e0a7e2a25621da82f22cc64d089eae05e753 by Miss Islington (bot)
in branch '3.10':
bpo-44426: Use of 'complex' as a C variable name confuses Sphinx; change it to
'num'. (GH-26744) (GH-26760)
https://gi
Mark Dickinson added the comment:
New changeset 686c6f303a6e9e54b50401be0ae3dc6aa2fcf05a by Miss Islington (bot)
in branch '3.9':
bpo-44426: Use of 'complex' as a C variable name confuses Sphinx; change it to
'num'. (GH-26744) (GH-26761)
https://gi
Mark Dickinson added the comment:
New changeset 7247f6f433846c6e37308a550e8e5eb6be379856 by Mark Dickinson in
branch 'main':
bpo-44426: Use of 'complex' as a C variable name confuses Sphinx; change it to
'num'. (GH-26744)
https://gi
Change by Mark Shannon :
--
pull_requests: +25344
pull_request: https://github.com/python/cpython/pull/26759
___
Python tracker
<https://bugs.python.org/issue44
Mark Shannon added the comment:
New changeset ab030d6f9d73e7f6c2213c2e308d1ceb04761485 by Sergey Fedoseev in
branch 'main':
bpo-38211: Clean up type_init() (GH-16257)
https://github.com/python/cpython/commit/ab030d6f9d73e7f6c2213c2e308d1ceb04761485
--
nosy: +Ma
Change by Mark Dickinson :
--
assignee: -> mark.dickinson
___
Python tracker
<https://bugs.python.org/issue44357>
___
___
Python-bugs-list mailing list
Un
Mark Dickinson added the comment:
I think this can be closed now.
--
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/i
Mark Dickinson added the comment:
New changeset 8d0b2ca493e236fcad8709a622c1ac8ad29c372d by Mark Dickinson in
branch '3.10':
[3.10] bpo-43475: Add what's new entry for NaN hash changes (GH-26725)
(GH-26743)
https://github.com/python/cpython/commit/8d0b2ca493e236fcad8709a62
Change by Mark Dickinson :
--
keywords: +patch
pull_requests: +25329
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/26744
___
Python tracker
<https://bugs.python.org/issu
Change by Mark Dickinson :
--
pull_requests: +25328
pull_request: https://github.com/python/cpython/pull/26743
___
Python tracker
<https://bugs.python.org/issue43
Mark Dickinson added the comment:
New changeset 1d10bf0bb9409a406c56b0de8870df998637fd0f by Mark Dickinson in
branch 'main':
bpo-43475: Add what's new entry for NaN hash changes (GH-26725)
https://github.com/python/cpython/commit/1d10bf0bb9409a406c56b0de8
Mark Dickinson added the comment:
As a test, gcc and clang both seem happy to treat this as valid C. I think
Sphinx is wrong to reject this.
mdickinson@mirzakhani Desktop % cat test.c
typedef struct {
double real;
double imag;
} Py_complex;
Py_complex _Py_c_neg(Py_complex complex
Mark Dickinson added the comment:
Hmm. It's probably not the best name, and we can certainly change it.
But I'm a bit confused by the error message: `complex` isn't a keyword in
either C or C++, so I'm not sure why Sphinx thinks it is.
--
n
New submission from Mark Final :
Hi,
We build Python 3.7 from source in-house for Windows, Linux and macOSX.
It's been noticed that 'dirty' has been appended to sys.version in our builds,
even though the source tree has not been modified from a git clone. This only
happen
Change by Mark Dickinson :
--
pull_requests: +25315
pull_request: https://github.com/python/cpython/pull/26725
___
Python tracker
<https://bugs.python.org/issue43
Mark Shannon added the comment:
This is going in the wrong direction.
Rather than add more complex instructions for use only by pattern matching, we
need to simplify the individual operations and re-use existing instructions.
That way pattern matching can benefit from the general performance
Mark Shannon added the comment:
What does "seem wrong" mean?
What exactly is the problem?
--
___
Python tracker
<https://bugs.python.org/issue44417>
___
__
Change by Mark Shannon :
--
pull_requests: +25306
pull_request: https://github.com/python/cpython/pull/26718
___
Python tracker
<https://bugs.python.org/issue44
Mark Shannon added the comment:
New changeset eecbc7c3900a7f40d8498b151db543a202c72f74 by Mark Shannon in
branch 'main':
bpo-44338: Port LOAD_GLOBAL to PEP 659 adaptive interpreter (GH-26638)
https://github.com/python/cpython/commit/eecbc7c3900a7f40d8498b151db543
Mark Dickinson added the comment:
> Does this change need to be mentioned in What's New?
Yes, I think so, given that it's a change to documented behavior. It's also
something that third party packages (like NumPy) potentially ne
Mark Dickinson added the comment:
@Serhiy: can this be closed again? Does GH-26679 need to be backported to the
3.10 branch?
--
___
Python tracker
<https://bugs.python.org/issue43
Mark Dickinson added the comment:
@realead
> This change makes life harder for people trying to get sane behavior with
> sets [...]
Yes, code that assumes that all NaNs have the same hash value will need to
change. But that doesn't seem unreasonable for objects that are already
Change by Mark Dickinson :
--
pull_requests: +25287
pull_request: https://github.com/python/cpython/pull/26703
___
Python tracker
<https://bugs.python.org/issue36
Change by Mark Dickinson :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Mark Dickinson added the comment:
New changeset 4a42cebf6dd769e2fa4e234a9e91093b3ad1cb63 by Mark Dickinson in
branch 'main':
bpo-44339: Fix math.pow corner case to comply with IEEE 754 (GH-26606)
https://github.com/python/cpython/commit/4a42cebf6dd769e2fa4e234a9e9109
Change by Mark Dickinson :
--
nosy: +mark.dickinson
___
Python tracker
<https://bugs.python.org/issue44400>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Mark Dickinson :
--
pull_requests: +25277
pull_request: https://github.com/python/cpython/pull/26690
___
Python tracker
<https://bugs.python.org/issue36
Mark Dickinson added the comment:
> That is, do we want NaNs to sort to the beginning of the list, or the end?
FWIW, NumPy chooses to sort NaNs to the end of the list:
https://numpy.org/doc/stable/reference/generated/numpy.sort.html
--
___
Pyt
Mark Dickinson added the comment:
> We should consider biting the bullet and revising the default NaN sort order.
If we went that route, I think we wouldn't need to consider payload or
identity. We could just do:
NaN < NaN -> False
NaN < non-NaN -&g
Change by Mark Gordon :
--
keywords: +patch
nosy: +msg555
nosy_count: 3.0 -> 4.0
pull_requests: +25251
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/26664
___
Python tracker
<https://bugs.python.org/i
Mark Dickinson added the comment:
> 2921 is the smallest integers for which math.sqrt(x) > x**0.5
This is platform-dependent. On my machine, for example, `math.sqrt(2921) ==
2921**0.5` returns `True`.
I don't see a lot of value in additional tests here; we're only wrappin
Mark Dickinson added the comment:
I can't reproduce this on my Mac laptop (using Python builds from MacPorts).
Numbers for both x**2 and x*x are fairly stable across Python 3.2 to Python
3.10. There's some variation, but nothing close to the same extent that Steven
is seeing.
H
Mark Dickinson added the comment:
All done; closing. Thank you for the contribution!
--
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/i
Mark Dickinson added the comment:
New changeset ac867f10b49322e25f34d2d8abd8e63c86834750 by Ajith Ramachandran in
branch 'main':
bpo-44357:Add `math.cbrt()` function: Cube Root (GH-26622)
https://github.com/python/cpython/commit/ac867f10b49322e25f34d2d8abd8e6
Change by Mark Dickinson :
--
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue44364>
___
___
Pyth
New submission from Mark Dickinson :
New changeset 90cd4330329a99e52f7141db5e0a469d30088e66 by Ajith Ramachandran in
branch 'main':
bpo-44364:Add non integral tests for `sqrt()` (#26625)
https://github.com/python/cpython/commit/90cd4330329a99e52f7141db5e0a46
Change by Mark Shannon :
--
pull_requests: +25226
pull_request: https://github.com/python/cpython/pull/26639
___
Python tracker
<https://bugs.python.org/issue44
Change by Mark Shannon :
--
keywords: +patch
pull_requests: +25225
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/26638
___
Python tracker
<https://bugs.python.org/issu
Change by Mark Shannon :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Mark Shannon added the comment:
Yes. Simpler is good.
I think it will also be better for performance:
In general, we don't know what X is in `from Y import X`. It could be a module
or anything else.
However, if we are accessing an attribute it is quite likely to be a module or
class
Mark Shannon added the comment:
It looks like I've been a bit unfair to the address sanitizer.
It does appear to produce incorrect locations sometimes, but that's not really
a false positive and the reports are generally useful.
--
Mark Shannon added the comment:
New changeset 54cb63863f19a7c64d9a3a5fd97bdfc0dd7ab374 by Mark Shannon in
branch 'main':
bpo-44348: Move trace-info to thread-state (GH-26623)
https://github.com/python/cpython/commit/54cb63863f19a7c64d9a3a5fd97bdf
Mark Shannon added the comment:
New changeset e117c0283705943189e6b1aef668a1f68f3f00a4 by Mark Shannon in
branch 'main':
bpo-44337: Port LOAD_ATTR to PEP 659 adaptive interpreter (GH-26595)
https://github.com/python/cpython/commit/e117c0283705943189e6b1aef668a1
Mark Dickinson added the comment:
See also #11986, which this report is essentially a duplicate of.
I think it may be time to implement `math.ieee754_total_order` (after suitable
bikeshedding about the name), so that one can do
sorted(my_list_of_floats, key=math.ieee754_total_order)
and
Mark Shannon added the comment:
No, this is done
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Mark Shannon added the comment:
This line seems to be responsible for most of the failures:
https://github.com/python/cpython/blob/main/Objects/frameobject.c#L985
Which does appear to be a true positive.
--
___
Python tracker
<ht
Mark Shannon added the comment:
What commit are you running that on?
--
___
Python tracker
<https://bugs.python.org/issue44363>
___
___
Python-bugs-list mailin
Change by Mark Dickinson :
--
nosy: +mark.dickinson
___
Python tracker
<https://bugs.python.org/issue44364>
___
___
Python-bugs-list mailing list
Unsubscribe:
Mark Dickinson added the comment:
If we *really* wanted to bikeshed on the name, back in 1991 Kahan wrote:
> Perhaps the last problem is the hardest: choosing the program's name. Ideally
> it should need no explanation, but a limitation upon its length may preclude
> that.
Mark Shannon added the comment:
I still get quite a few failures on the main branch.
It seems like ceval.c:1600 upsets the sanitizer, at least for gcc.
There isn't anything wrong with that line, but as I plan to change it anyway I
guess it doesn't matter:
https://github.com/pyth
Change by Mark Shannon :
--
pull_requests: +25209
pull_request: https://github.com/python/cpython/pull/26624
___
Python tracker
<https://bugs.python.org/issue44
Mark Dickinson added the comment:
@Victor: Yep, the name is pretty standard. Not just C, but JavaScript, Java, R,
and likely a lot of other languages that I haven't checked.
--
___
Python tracker
<https://bugs.python.org/is
Change by Mark Shannon :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Mark Dickinson added the comment:
There's also a decent chance that a libm implementation of cbrt will be
correctly rounded, while `x**(1/3)` is highly unlikely to be so.
--
___
Python tracker
<https://bugs.python.org/is
Mark Dickinson added the comment:
> Perhaps because it is so easy to write x**(1/3), and if you want a real root
> of negative argument, it is -(-x)**(1/3).
I consider `x**(1/3)` to be a trap and a bug magnet: many people won't realise
that the correct spelling for their s
Mark Shannon added the comment:
If I run the following on main
./configure --with-address-sanitizer
make clean
make -j12 test
I get 22 failures.
So something is broken.
test_lib2to3 does fork; at least it does when I run it under gdb
Change by Mark Shannon :
--
nosy: +Mark.Shannon
nosy_count: 9.0 -> 10.0
pull_requests: +25208
pull_request: https://github.com/python/cpython/pull/26623
___
Python tracker
<https://bugs.python.org/issu
New submission from Mark Shannon :
Running the buildbot for https://github.com/python/cpython/pull/26595
results in failures:
https://buildbot.python.org/all/#/builders/581/builds/63
Which claim that the address calculation in `LOAD_ATTR_MODULE` is out of bounds.
The tests pass with an added
Mark Dickinson added the comment:
BTW, to forestall questions about adding cmath.cbrt: it's not obvious how to
extend the real cube root function to a complex cube root, so it might make
sense to wait for actual use-cases before doing so.
The issue is that the most natural way to def
Mark Dickinson added the comment:
+1. This is part of C99, so if it's also supported by Visual Studio, then this
seems like a no-brainer. If it's _not_ also supported by Visual Studio, or if
there are implementations that have serious numerical problems (as was the case
with fma)
Mark Dickinson added the comment:
> I was surprised to realize `complex` doesn't accept '2 + 3j'
See previous discussion in #9574.
--
nosy: +mark.dickinson
___
Python tracker
<https://bugs.
Change by Mark Dickinson :
--
keywords: +patch
pull_requests: +25189
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/26606
___
Python tracker
<https://bugs.python.org/issu
Mark Dickinson added the comment:
[Dennis]
> I think the relevant property is that the exponent is not an integer
Yep: the delegation to complex pow kicks in after handling infinities and nans,
and only for strictly negative base (-0.0 doesn't count as negative for this
purpose)
Mark Shannon added the comment:
Pablo,
Is there a bpo issue for the buildbot failures on Windows?
The failures I've been seeing are C stack overflows.
Long term, I expect to fix it by decoupling the C and Python stacks.
In the short term I have a couple of changes that might get it wo
Change by Mark Shannon :
--
keywords: +patch
pull_requests: +25178
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/26595
___
Python tracker
<https://bugs.python.org/issu
Mark Dickinson added the comment:
> What about math.pow(0.0, -1.0)? Should it return -inf or raise
> ZeroDivisionError?
Neither: it should raise ValueError, as it does now. This matches IEEE 754
along with the math module's mapping of IEEE 754 floating-point exceptions to
Python
Change by Mark Dickinson :
--
components: +Library (Lib)
type: -> enhancement
___
Python tracker
<https://bugs.python.org/issue44339>
___
___
Python-bugs-lis
New submission from Mark Dickinson :
For floats x and y, x ** y and math.pow(x, y) mostly agree. There are three
points of difference:
1. if x is finite and negative and y is finite and non-integral, then x ** y
returns a complex number, while math.pow(x, y) raises ValueError
2. for cases
New submission from Mark Shannon :
Port the implementation of LOAD_GLOBAL to the new adaptive interpreter
Once this and https://bugs.python.org/issue44337 are implemented we can remove
the old opcache.
--
messages: 395272
nosy: Mark.Shannon
priority: normal
severity: normal
status
New submission from Mark Shannon :
Port the implementation of LOAD_ATTR to the new adaptive interpreter
--
messages: 395271
nosy: Mark.Shannon
priority: normal
severity: normal
status: open
title: Port LOAD_ATTR to adaptive interpreter
___
Python
Mark Shannon added the comment:
New changeset 001eb520b5757294dc455c900d94b7b153de6cdd by Mark Shannon in
branch 'main':
bpo-44187: Quickening infrastructure (GH-26264)
https://github.com/python/cpython/commit/001eb520b5757294dc455c900d94b7
Mark Shannon added the comment:
Basic blocks have only a single exit, at the end.
https://en.wikipedia.org/wiki/Basic_block
If the devguide says otherwise it is wrong.
--
___
Python tracker
<https://bugs.python.org/issue42
Change by Mark Dickinson :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> enhancement
___
Python tracker
<https://bugs.python
Mark Dickinson added the comment:
New changeset 89e50ab36fac6a0e7f1998501f36fcd2872a6604 by Sergey B Kirpichev in
branch 'main':
bpo-44258: support PEP 515 for Fraction's initialization from string (GH-26422)
https://github.com/python/cpython/commit/89e50ab36fac6a0e7f1998501
Change by Mark Dickinson :
--
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue44332>
___
___
Python-bugs-list mailing list
Unsubscrib
Mark Bell added the comment:
> Instead, the discussion was focused on removing *all* empty strings from the
> result.
I imagine that the discussion focussed on this since this is precisely what
happens when sep=None. For example, 'a b c '.split() == ['a'
Mark Bell added the comment:
Andrei: That is a very interesting observation, thank you for pointing it out.
I guess your example / argument also currently applies to whitespace separation
too. For example, if we have a whitespace separated string with contents:
col1 col2 col3
a b c
x y z
Mark Dickinson added the comment:
Ah, I think I see: you want a function that turns the string "foo\bar" into
"foo\\bar". Even if this were a good idea, I don't think it's feasible to do it
in a non-surprising way.
For example, given such a function f, wha
Mark Dickinson added the comment:
Sorry, I missed the definition of f in the last message. Trying again:
>>> def f(x): return x
...
>>> f(r'foo\bar') == 'foo\\bar'
True
--
___
Python tracke
Mark Dickinson added the comment:
> But there's no builtin function for r'foo\bar' that gives you 'foo\\bar'.
I'm confused about what's being requested here. r'foo\bar' and 'foo\\bar' are
different source code representations of
Change by Mark Shannon :
--
pull_requests: +25111
pull_request: https://github.com/python/cpython/pull/26516
___
Python tracker
<https://bugs.python.org/issue44
Mark Shannon added the comment:
New changeset 937cebc93b4922583218e0cbf0a9a14705a595b2 by Mark Shannon in
branch 'main':
bpo-44298: Fix line numbers for early exits in with statements. (GH-26513)
https://github.com/python/cpython/commit/937cebc93b4922583218e0cbf0a9a1
Change by Mark Shannon :
--
keywords: +patch
pull_requests: +25109
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/26513
___
Python tracker
<https://bugs.python.org/issu
Mark Shannon added the comment:
Why this occurs:
with cm:
A
break
translates to something like:
ex = cm.__exit__; cm.__enter__() # with cm
A
ex(...)
goto loop_end # break
So, the break is traced after the exit call.
However, this doesn't seem consistent wit
Mark Shannon added the comment:
For context, this behavior was introduced in https://bugs.python.org/issue43933
--
___
Python tracker
<https://bugs.python.org/issue44
Change by Mark Dickinson :
--
resolution: -> wont fix
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue44267>
___
___
Mark Dickinson added the comment:
> Regarding mail thread: I don't think that following the PEP will
slow down string conversion.
Sorry, I just don't think it's worth re-opening this discussion; Stefan Krah
had good reasons (not just speed) to avoid implementing a precise
Mark Dickinson added the comment:
> Well, then I something misunderstood in PEP 0
Yep, you're absolutely right. I should have said "after the PEP is final", not
"after the PEP is accepted". PEP 515 was marked final on April 28th, 2017.
> The current beha
Mark Dickinson added the comment:
> If this is not a bug - it should be adjusted
Standards Track PEPs are historical documents; it's quite common that an actual
implementation ends up diverging from a PEP in small ways after the PEP is
accepted, and we don't usually do post-h
Mark Dickinson added the comment:
There was some discussion of this on the python-dev mailing list at the time -
see https://mail.python.org/pipermail/python-dev/2016-March/143556.html and the
surrounding thread. It's probably best left
Mark Dickinson added the comment:
> How about '1_/_2'?
I'd rather keep it consistent with the rules for int: that is, if I split on
`'/'`, I'd expect the pieces to be parseable by `int`.
As for spaces around the `/`,
Mark Dickinson added the comment:
Agreed. Thank you!
--
stage: -> resolved
status: pending -> closed
___
Python tracker
<https://bugs.python.org/i
Change by Mark Dickinson :
--
title: Python -> Install Include/internal/ header files
versions: +Python 3.8
___
Python tracker
<https://bugs.python.org/issu
Change by Mark Dickinson :
--
Removed message: https://bugs.python.org/msg394672
___
Python tracker
<https://bugs.python.org/issue35296>
___
___
Python-bug
Change by Mark Dickinson :
--
Removed message: https://bugs.python.org/msg394671
___
Python tracker
<https://bugs.python.org/issue35296>
___
___
Python-bug
Change by Mark Shannon :
--
pull_requests: +25027
pull_request: https://github.com/python/cpython/pull/26432
___
Python tracker
<https://bugs.python.org/issue44
Change by Mark Dickinson :
--
nosy: +mark.dickinson
___
Python tracker
<https://bugs.python.org/issue32596>
___
___
Python-bugs-list mailing list
Unsubscribe:
Mark Dickinson added the comment:
You say "for testing purposes only", but I'd add "for debugging" to the list of
use-cases, having just spent a few hours trying to understand why I was getting
a particular PySide2 "Internal C++ object already deleted."
Mark Shannon added the comment:
New changeset f8a95df84bcedebc0aa7132b3d1a4e8f000914bc by Mark Shannon in
branch 'main':
bpo-44206: Add a version number to dictionary keys (GH-26333)
https://github.com/python/cpython/commit/f8a95df84bcedebc0aa7132b3d1a4e
Mark Dickinson added the comment:
+1 to the idea: `Decimal`, `float`, `complex` and `int` all understand the
underscores; there's no good reason for `Fraction` not to.
So yes please to the PR. There should be tests that check that only the
underscore rules allowed by `int` and the o
901 - 1000 of 12241 matches
Mail list logo