Antoine Pitrou added the comment:
FWIW, I have the roles "User,Developer,Coordinator" according to my user page.
You (Terry) have the roles "User,Developer".
Perhaps the "Coordinator" role explains I see things you don't see (it's a
better explanation tha
Antoine Pitrou added the comment:
Terry, the list of roles in https://bugs.python.org/user11089 is empty, which I
take it to mean that Stefan doesn't have any access rights on this issue
tracker (except to read issues). Other users all seem to have the "User&
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Antoine Pitrou added the comment:
New changeset ba251c2ae6654bfc8abd9d886b219698ad34ac3c by Antoine Pitrou in
branch 'master':
bpo-43356: Allow passing a signal number to interrupt_main() (GH-24755)
https://github.com/python/cpython/commit/ba251c2ae6654bfc8abd9d886b219698ad34ac3c
Change by Antoine Pitrou :
--
keywords: +patch
pull_requests: +23559
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/24792
___
Python tracker
<https://bugs.python.org/issu
Change by Antoine Pitrou :
--
pull_requests: +23546
pull_request: https://github.com/python/cpython/pull/24755
___
Python tracker
<https://bugs.python.org/issue43
Antoine Pitrou added the comment:
@mattip, see issue43422
--
assignee: skrah ->
___
Python tracker
<https://bugs.python.org/issue41324>
___
___
Python-
Antoine Pitrou added the comment:
Stefan wrote me that the changes which complete this update are here:
https://github.com/python/cpython/compare/master...skrah:libmpdec-2.5.1
I can turn this diff into a PR but first I want to gather feedback here. Mark,
Serhiy, Raymond, what do you say
New submission from Antoine Pitrou :
Stefan Krah (who doesn't have access rights here, and is the author of the C
_decimal module) asked me to transmit me this request:
"""
The capsule API does not meet my testing standards, since I've focused
on the upstream mpdecimal in
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Antoine Pitrou :
--
pull_requests: +23534
pull_request: https://github.com/python/cpython/pull/24762
___
Python tracker
<https://bugs.python.org/issue43
Antoine Pitrou added the comment:
New changeset 68245b7a1030287294c65c298975ab9026543fd2 by Antoine Pitrou in
branch 'master':
bpo-43406: Fix possible race condition where ``PyErr_CheckSignals`` tries to
execute a non-Python signal handler (GH-24756)
https://github.com/python/cpython/commit
Antoine Pitrou added the comment:
Turns out this didn't fix all possible situations. See bpo-43406 for an updated
patch.
--
___
Python tracker
<https://bugs.python.org/issue23
Change by Antoine Pitrou :
--
resolution: -> duplicate
stage: test needed -> resolved
status: open -> closed
superseder: -> Possible race condition between signal catching and
signal.signal
___
Python tracker
<https://bugs.python
Change by Antoine Pitrou :
--
nosy: +neologix
___
Python tracker
<https://bugs.python.org/issue43406>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Antoine Pitrou :
--
keywords: +patch
pull_requests: +23527
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/24756
___
Python tracker
<https://bugs.python.org/issu
Antoine Pitrou added the comment:
Here is a reproducer:
https://gist.github.com/pitrou/e5a566e644730516b51de71145c5ea06
If you execute it, it will fail after a few iterations:
sig 2
sig 2
sig 2
Traceback (most recent call last):
File "/home/antoine/cpython/default/setinterrupt.py"
New submission from Antoine Pitrou :
We can receive signals (at the C level, in trip_signal() in signalmodule.c)
while signal.signal is being called to modify the corresponding handler. Later
when PyErr_CheckSignals() is called to handle the given signal, the handler may
be a non-callable
Change by Antoine Pitrou :
--
keywords: +patch
pull_requests: +23526
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/24755
___
Python tracker
<https://bugs.python.org/issu
Change by Antoine Pitrou :
--
nosy: +neologix, vstinner
___
Python tracker
<https://bugs.python.org/issue43356>
___
___
Python-bugs-list mailing list
Unsub
Antoine Pitrou added the comment:
I have no opinion about *adding* a struct, but we shouldn't remove the existing
array of pointers, or this will needlessly break compatibility for existing
users of the C API.
--
nosy: +pitrou
___
Python tracker
New submission from Antoine Pitrou :
PyErr_SetInterrupt() is useful if you want to simulate the effect of a SIGINT.
It would be helpful to provide a similar primitive for other signal numbers,
e.g. `PyErr_SetInterruptEx(int signum)`.
--
components: C API
messages: 387877
nosy: pitrou
Change by Antoine Pitrou :
--
resolution: -> later
stage: needs patch -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/
Antoine Pitrou added the comment:
I don't think so.
--
___
Python tracker
<https://bugs.python.org/issue1152248>
___
___
Python-bugs-list mailing list
Unsub
New submission from Antoine Pitrou :
While this makes sense given the exposed functionality, the reader may be wary
of calling PyErr_SetInterrupt from a C signal handler if it's not explicitly
documented as signal-safe.
--
assignee: docs@python
components: C API, Documentation
Change by Antoine Pitrou :
--
nosy: +aeros
___
Python tracker
<https://bugs.python.org/issue41567>
___
___
Python-bugs-list mailing list
Unsubscribe:
Antoine Pitrou added the comment:
Thank you @erlendaasland !
--
components: +Extension Modules -C API, Subinterpreters
nosy: +pitrou
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> enhancement
__
Antoine Pitrou added the comment:
Oops, sorry, I hadn't noticed this was a catch-all issue. Reopening.
--
resolution: fixed ->
stage: resolved ->
status: closed -> open
___
Python tracker
<https://bugs.python.or
Change by Antoine Pitrou :
--
nosy: +gregory.p.smith
___
Python tracker
<https://bugs.python.org/issue42097>
___
___
Python-bugs-list mailing list
Unsubscribe:
Antoine Pitrou added the comment:
At some point, pickle protocols 0 and 1 should be deprecated and slated for
removal. Protocol 2 is 17 years ago already, and protocol 3 has become the
default in Python 3.
That would probably be a better use of contributor time than trying to make
heap
Antoine Pitrou added the comment:
FTR, no formal description of the algorithm seems to have been published, but
the source code is amply commented:
https://github.com/lemire/fast_double_parser/blob/master/include/fast_double_parser.h
--
nosy: +pitrou
Change by Antoine Pitrou :
--
priority: normal -> low
___
Python tracker
<https://bugs.python.org/issue41310>
___
___
Python-bugs-list mailing list
Unsubscrib
Change by Antoine Pitrou :
--
resolution: -> duplicate
stage: -> resolved
status: open -> closed
superseder: -> micro-optimization: increase our float parsing speed by Nx
___
Python tracker
<https://bugs.python
Antoine Pitrou added the comment:
Oh, thank you.
--
___
Python tracker
<https://bugs.python.org/issue42081>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Antoine Pitrou :
This is more of a "could be nice" entry if there are any interested core
developers.
A new algorithm and library was recently published for faster parsing of double
numbers. It claims rigorous accuracy and the performance is
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Antoine Pitrou :
--
nosy: +eric.snow
___
Python tracker
<https://bugs.python.org/issue37881>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Antoine Pitrou :
--
versions: -Python 3.7
___
Python tracker
<https://bugs.python.org/issue41066>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Antoine Pitrou :
--
pull_requests: +20991
pull_request: https://github.com/python/cpython/pull/21864
___
Python tracker
<https://bugs.python.org/issue41
Antoine Pitrou added the comment:
New changeset 0eb9deb4a62e6d9daa82bc2f67d1075864ca8ece by Srinivas Reddy
Thatiparthy (శ్రీనివాస్ రెడ్డి తాటిపర్తి) in branch 'master':
bpo-41066: Update the comparison section for os vs pathlib (GH-21261)
https://github.com/python/cpython/commit
Antoine Pitrou added the comment:
> I'm going to rename this bug to focus only on issue #1. I think we can accept
> a PR adding two new macros. I would suggest calling them:
> - PyDateTime_DATE_GET_TZINFO
> - PyDateTime_TIME_GET_TZINFO
+1 for this.
--
n
Change by Antoine Pitrou :
--
stage: -> needs patch
type: -> behavior
versions: +Python 3.10
___
Python tracker
<https://bugs.python.org/issue39604>
___
__
Antoine Pitrou added the comment:
I probably won't try it out explicitly (it's basically cumbersome for us to
test with non-release Pythons, because of our dependencies to Numpy and
Cython), but reviewing the API should be enough anyway
Change by Antoine Pitrou :
--
nosy: +serhiy.storchaka, vstinner
___
Python tracker
<https://bugs.python.org/issue41208>
___
___
Python-bugs-list mailin
Antoine Pitrou added the comment:
Thanks for the clarification. I agree this does not seem to be a very big
deal, if slightly annoying for the packager who will have to deal with it.
--
___
Python tracker
<https://bugs.python.org/issue40
Antoine Pitrou added the comment:
Hmm, I'm only taking notice of this comment thread now.
(sorry, but due to spam filtering issues I only receive bpo e-mail
notifications intermittently... and that's despite having tried two separate
e-mail providers which otherwise give me no problems
Antoine Pitrou added the comment:
No strong feelings on this, but the OpenSSL runtime is not always packaged by a
Linux distribution. (macOS, Windows and Anaconda come to mind)
If one wants to retain the setter facility, one could raise a RuntimeWarning if
the user *lowers* the actual
Change by Antoine Pitrou :
--
versions: +Python 3.8, Python 3.9 -Python 2.7, Python 3.4, Python 3.5, Python
3.6
___
Python tracker
<https://bugs.python.org/issue29
New submission from Antoine Pitrou :
A number of codecs raise bare UnicodeError, rather than
Unicode{Decode,Encode}Error. Example:
File
"/home/antoine/miniconda3/envs/pyarrow/lib/python3.7/encodings/utf_16.py", line
67, in _buffer_decode
raise UnicodeError("UTF
Change by Antoine Pitrou :
--
nosy: -pitrou
___
Python tracker
<https://bugs.python.org/issue15655>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> behavior
versions: +Python 3.7, Python 3.8, Python 3.9
___
Python tracker
<https://bugs.python
Change by Antoine Pitrou :
--
pull_requests: +19989
pull_request: https://github.com/python/cpython/pull/20793
___
Python tracker
<https://bugs.python.org/issue40
Change by Antoine Pitrou :
--
nosy: +pitrou
nosy_count: 3.0 -> 4.0
pull_requests: +19988
pull_request: https://github.com/python/cpython/pull/20792
___
Python tracker
<https://bugs.python.org/issu
Antoine Pitrou added the comment:
The workaround that Cython had to added for this (temporarily enable
Py_TPFLAGS_HEAPTYPE when calling PyType_Ready()) is fragile. It would be nice
to rethink the approach here, or disable the check altogether.
(or perhaps, expose another C API function
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.7, Python 3.8
___
Python tracker
<https://bugs.python.or
Change by Antoine Pitrou :
--
pull_requests: +19162
pull_request: https://github.com/python/cpython/pull/19844
___
Python tracker
<https://bugs.python.org/issue39
Change by Antoine Pitrou :
--
pull_requests: +19161
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/19843
___
Python tracker
<https://bugs.python.org/issu
Antoine Pitrou added the comment:
New changeset 8727664557cd44dcd00612ccba816942e8f885ab by Dong-hee Na in branch
'master':
bpo-32494: Use gdbm_count for dbm_length if possible (GH-19814)
https://github.com/python/cpython/commit/8727664557cd44dcd00612ccba816942e8f885ab
--
nosy
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Antoine Pitrou added the comment:
The TextIOWrapper uses an underlying BufferedWriter, which is thread-safe (and
therefore has an internal lock).
--
___
Python tracker
<https://bugs.python.org/issue40
Antoine Pitrou added the comment:
I looked at the change and it seemed ok to me. Perhaps Thomas can give it a
look too.
--
assignee: aeros -> vstinner
___
Python tracker
<https://bugs.python.org/issu
Antoine Pitrou added the comment:
With the same traceback and error message?
--
___
Python tracker
<https://bugs.python.org/issue39995>
___
___
Python-bug
Antoine Pitrou added the comment:
Related issue:
https://bugs.python.org/issue40399
"""
IO streams locking can be broken after fork() with threads
"""
--
___
Python tracker
<
Antoine Pitrou added the comment:
Well, as the documentation states, `QueueListener.start` """starts up a
background thread to monitor the queue for LogRecords to process""" :-)
--
___
Python tracker
<h
Antoine Pitrou added the comment:
I can reproduce on Ubuntu 18.04 with git master.
Here is a better example which clearly shows the issue:
https://gist.github.com/pitrou/d9784d5ec679059cd02fce4b38ea2fa6
After a few runs, you'll see that the child Process hangs when trying to flush
Antoine Pitrou added the comment:
How about the following (untested):
diff --git a/Lib/concurrent/futures/process.py
b/Lib/concurrent/futures/process.py
index 8e9b69a8f0..c0c2eb3032 100644
--- a/Lib/concurrent/futures/process.py
+++ b/Lib/concurrent/futures/process.py
@@ -66,23 +66,29
Antoine Pitrou added the comment:
You're right that a Py_DECREF is missing there. However, it seems unlikely
that this is triggering the test failure, because `PyList_New(1)` will
practically never fail in normal conditions (and even making it deliberately
fail would be quite difficult
Antoine Pitrou added the comment:
I don't know if a try..except block is the best solution, but feel free to
submit a PR and we can iterate on that :-)
--
stage: -> needs patch
___
Python tracker
<https://bugs.python.org/issu
Antoine Pitrou added the comment:
I think we would accept a PR since it would probably be trivial, but someone
has to submit it :-)
--
___
Python tracker
<https://bugs.python.org/issue37
Antoine Pitrou added the comment:
If I believe what Victor wrote about:
"""
The implementation of random.Random, random.SystemRandom and
random.Random subclasses are not affected by this change.
"""
then I don't understand how the API is changed. IIUC, users su
Antoine Pitrou added the comment:
The current behaviour is by design. I would not mind adding an option to
control it, though.
If you are new to Python development and want to submit a patch or PR, I
recommend reading the Developer's Guide:
https://devguide.python.org
Change by Antoine Pitrou :
--
keywords: +patch
nosy: +pitrou
nosy_count: 1.0 -> 2.0
pull_requests: +18957
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/19539
___
Python tracker
<https://bugs.python.org/i
Antoine Pitrou added the comment:
Another possibility yet would be:
typedef struct {
PyObject_VAR_HEAD
Py_hash_t ob_shash;
char ob_sval;
} PyBytesObject;
#define PyBytes_AS_STRING(op) (assert(PyBytes_Check(op)), \
&(((PyBytesObject *)(op))->
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
versions: -Python 3.7
___
Python tracker
<https://bugs.python.or
Antoine Pitrou added the comment:
New changeset 887ff8e37e238fbce18c647e588283904f38ab24 by Antoine Pitrou in
branch '3.8':
[3.8] bpo-40330: Fix utf-8 size check in ShareableList (GH-19606) (GH-19625)
https://github.com/python/cpython/commit/887ff8e37e238fbce18c647e588283904f38ab24
Change by Antoine Pitrou :
--
pull_requests: +18953
pull_request: https://github.com/python/cpython/pull/19625
___
Python tracker
<https://bugs.python.org/issue40
Antoine Pitrou added the comment:
Wow. I see, thank you.
--
___
Python tracker
<https://bugs.python.org/issue40255>
___
___
Python-bugs-list mailing list
Unsub
Antoine Pitrou added the comment:
> We also have the real world app that is Instagram.
I don't think Instagram is a single app. What is Python used for at Instagram?
--
___
Python tracker
<https://bugs.python.org/issu
Antoine Pitrou added the comment:
> Benchmarking already showed that the branching version is faster.
But micro-benchmarks may tell you things which are not true in the real-world
(for example an easily-predicted branch).
--
___
Python trac
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Antoine Pitrou added the comment:
New changeset 8aea4b3605059e243f1827d9328d6fc8d698c0a7 by Tim Hoffmann in
branch 'master':
bpo-40148: Add PurePath.with_stem() (GH-19295)
https://github.com/python/cpython/commit/8aea4b3605059e243f1827d9328d6fc8d698c0a7
Change by Antoine Pitrou :
--
keywords: +patch
pull_requests: +18940
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/19606
___
Python tracker
<https://bugs.python.org/issu
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Antoine Pitrou added the comment:
New changeset c8f1715283ec51822fb37a702bf253cbac1af276 by Thomas Krennwallner
in branch 'master':
bpo-38891: avoid quadratic item access performance of ShareableList (GH-18996)
https://github.com/python/cpython/commit/c8f1715283ec51822fb37a702bf253cbac1af276
New submission from Antoine Pitrou :
The size check is done before encoding to utf-8...
--
___
Python tracker
<https://bugs.python.org/issue40330>
___
___
Pytho
Change by Antoine Pitrou :
--
assignee: pitrou
components: Library (Lib)
nosy: pitrou
priority: normal
severity: normal
stage: needs patch
status: open
title: ShareableList size guard incorrect for str data
type: behavior
versions: Python 3.7, Python 3.8, Python 3.9
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Antoine Pitrou :
--
versions: +Python 3.9 -Python 3.8
___
Python tracker
<https://bugs.python.org/issue38891>
___
___
Python-bugs-list mailin
Change by Antoine Pitrou :
--
versions: +Python 3.9
___
Python tracker
<https://bugs.python.org/issue35306>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Antoine Pitrou :
--
versions: +Python 3.9
___
Python tracker
<https://bugs.python.org/issue33898>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> enhancement
___
Python tracker
<https://bugs.python
Antoine Pitrou added the comment:
New changeset 5b1d9184bb0e34391637c06bc7651fb6de8a6240 by Barney Gale in branch
'master':
bpo-39894: Route calls from pathlib.Path.samefile() to os.stat() via the path
accessor (GH-18836)
https://github.com/python/cpython/commit
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> enhancement
___
Python tracker
<https://bugs.python
Antoine Pitrou added the comment:
New changeset c746c4f353510a17683a49ed7f90ffaae664ff7b by Barney Gale in branch
'master':
bpo-39897: Remove needless `Path(self.parent)` call, which makes `is_mount()`
misbehave in `Path` subclasses. (GH-18839)
https://github.com/python/cpython/commit
Change by Antoine Pitrou :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Antoine Pitrou added the comment:
New changeset 75a3378810bab03949ad9f653f78d933bdf3879c by Antoine Pitrou in
branch 'master':
bpo-40282: Allow random.getrandbits(0) (GH-19539)
https://github.com/python/cpython/commit/75a3378810bab03949ad9f653f78d933bdf3879c
Change by Antoine Pitrou :
--
nosy: -pitrou
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> enhancement
___
Python tracker
<https://bugs.python
Antoine Pitrou added the comment:
New changeset 22386bb4ef740ee92d34c87b8cb90d681423a853 by Barney Gale in branch
'master':
bpo-39901: Move `pathlib.Path.owner()` and `group()` implementations into the
path accessor. (GH-18844)
https://github.com/python/cpython/commit
Antoine Pitrou added the comment:
Hmm, it looks like I lost my head from 10 years ago :-S
Sorry for the noise, everyone.
--
resolution: -> works for me
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bug
101 - 200 of 16564 matches
Mail list logo
