Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
nosy_count: 8.0 -> 9.0
pull_requests: +25593
pull_request: https://github.com/python/cpython/pull/27033
___
Python tracker
<https://bugs.python.org/issu
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue39759>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
nosy_count: 4.0 -> 5.0
pull_requests: +21279
pull_request: https://github.com/python/cpython/pull/21763
___
Python tracker
<https://bugs.python.org/issu
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue35276>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue26834>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +20793
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/21652
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
In [191]: import dataclasses, pydoc
In [192]: @dataclass
...: class C:
...: pass
...:
In [193]: print(pydoc.render_doc(C))
Python Library Documentation: class C in module __main__
class C(builtins.object)
| C() -> None
|
|
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +20336
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/21179
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
Currently patlib.Path.iterdir() filters out '.' and '..'.
It's unneeded since patlib.Path.iterdir() uses os.listdir() under the hood,
which returns neither '.' nor '..'.
https://docs.python.org/3/library/os.html#os.listdir
--
components: Library
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
nosy_count: 3.0 -> 4.0
pull_requests: +19338
pull_request: https://github.com/python/cpython/pull/15659
___
Python tracker
<https://bugs.python.org/issu
Change by Sergey Fedoseev :
--
pull_requests: +17021
pull_request: https://github.com/python/cpython/pull/17539
___
Python tracker
<https://bugs.python.org/issue27
Change by Sergey Fedoseev :
--
pull_requests: +16894
pull_request: https://github.com/python/cpython/pull/17413
___
Python tracker
<https://bugs.python.org/issue37
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue38524>
___
___
Python-bugs-list mailing list
Unsubscribe:
Sergey Fedoseev added the comment:
issue38524 is related.
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue38517>
___
___
Python-bugs-list m
Sergey Fedoseev added the comment:
These warnings are caused by
https://github.com/python/cpython/commit/c6734ee7c55add5fdc2c821729ed5f67e237a096.
I'd fix them, but I'm not sure if we are going to restore CHECK_SMALL_INT()
¯\_(ツ)_/¯
--
nosy: +sir-sigurd
Sergey Fedoseev added the comment:
> If you care of _PyLong_Copy() performance, you should somehow manually inline
> _PyLong_New() inside _PyLong_Copy().
It doesn't solve this:
> We could add a function that bypass that check, but in LTO build
> PyObject_MALLOC(
Sergey Fedoseev added the comment:
FWIW I proposed to add Py_ASSUME() macro that uses __builtin_unreachable() in
bpo-38147.
--
___
Python tracker
<https://bugs.python.org/issue38
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +15852
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/16257
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
I wrote patch that cleans up type_init():
1. Removes conditions already checked by assert()
2. Removes object_init() call that effectively creates an empty tuple and
checks that this tuple is empty
--
messages: 352710
nosy: sir-sigurd
priority
Sergey Fedoseev added the comment:
Also quote from Py_UNREACHABLE() doc:
> Use this in places where you might be tempted to put an assert(0) or abort()
> call.
https://github.com/python/cpython/commit/6b519985d23bd0f0bd072b5d5d5f2c60a81a19f2
does exactly that, it replaces as
Sergey Fedoseev added the comment:
I believe that the problem is caused by the change in Py_UNREACHABLE()
(https://github.com/python/cpython/commit/3ab61473ba7f3dca32d779ec2766a4faa0657923).
Before the mentioned commit Py_UNREACHABLE() was an expression, now it's a
block. Py_UNREACHABLE
New submission from Sergey Fedoseev :
GCC (along with Clang and ICC) has __builtin_unreachable() and MSVC has
__assume() builtins, that can be used to optimize out unreachable conditions.
So we could add macro like this:
#ifdef Py_DEBUG
# define Py_ASSUME(cond) (assert(cond))
#else
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +15517
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15274
___
Python tracker
<https://bugs.python.org/issu
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +15471
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/14838
___
Python tracker
<https://bugs.python.org/issu
Sergey Fedoseev added the comment:
I use GCC 9.2.
--
___
Python tracker
<https://bugs.python.org/issue38015>
___
___
Python-bugs-list mailing list
Unsubscribe:
Sergey Fedoseev added the comment:
I added similar patch that replaces get_small_int() with macro version, since
it also induces unnecessary casts and makes machine code less efficient.
Example assembly can be checked at https://godbolt.org/z/1SjG3E.
This change produces tiny
Change by Sergey Fedoseev :
--
pull_requests: +15372
pull_request: https://github.com/python/cpython/pull/15718
___
Python tracker
<https://bugs.python.org/issue38
New submission from Sergey Fedoseev :
This patch simplifies fast path for floats that fit into C long and moves it
from float.__trunc__ to PyLong_FromDouble().
+-+-+--+
| Benchmark | long-from-float-ref | long-from
Sergey Fedoseev added the comment:
> map() does not have anything special.
Just for the reference, in Python 2.7 map() has the same behavior and it caused
many problems for me and other people working with/developing Django.
--
___
Python trac
Sergey Fedoseev added the comment:
Also using this example class:
In [5]: iter(Iterable('one'))
---
TypeError Traceback (most recent call last)
in ()
> 1 iter(Iterable('
Sergey Fedoseev added the comment:
Maybe it's not clear from description, but traceback only show the line with
zip(), so it doesn't help at localizing the source of exception at all.
You only see that 'argument #N must support iteration', but that argument has
__iter__() i.e. it supports
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +15268
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15592
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
zip() shadows TypeError raised in __iter__() of source iterable:
In [21]: class Iterable:
...: def __init__(self, n):
...: self.n = n
...: def __iter__(self):
...: return iter(range(self.n))
...:
In [22
New submission from Sergey Fedoseev :
In [3]: help(zip)
class zip(object)
| zip(*iterables) --> zip object
|
| Return a zip object whose .__next__() method returns a tuple where
| the i-th element comes from the i-th iterable argument. The .__next__()
| method continues un
New submission from Sergey Fedoseev :
In [8]: help(sys.float_info)
...
|
| dig
| DBL_DIG -- digits
|
This is not very helpful,
https://docs.python.org/3/library/sys.html#sys.float_info is more verbose, so
probably docstrings should be updated from where.
--
assignee
Sergey Fedoseev added the comment:
These last results are invalid :-)
I thought that I was checking _PyLong_FromUnsignedChar() on top of GH-15192,
but that wasn't true. So the correct results for LTO build are:
$ python -m perf timeit -s "from collections import deque; consume =
Sergey Fedoseev added the comment:
Previous benchmarks results were obtained with non-LTO build.
Here are results for LTO build:
$ python -m perf timeit -s "from itertools import repeat; _len = repeat(None,
0).__length_hint__" "_len()" --compare-to=../cpython-ma
Sergey Fedoseev added the comment:
$ gcc -v 2>&1 | grep 'gcc version'
gcc version 8.3.0 (Debian 8.3.0-19)
using ./configure --enable-optimizations --with-lto
$ python -m perf timeit -s "from collections import deque; consume =
deque(maxlen=0).extend; b = bytes(2**20)" "
New submission from Sergey Fedoseev :
PyLong_As*() functions have a lot of duplicated code, creating draft PR with
attempt to deduplicate them.
--
components: Interpreter Core
messages: 350367
nosy: sir-sigurd
priority: normal
severity: normal
status: open
title: refactor PyLong_As
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +15152
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15457
___
Python tracker
<https://bugs.python.org/issu
Change by Sergey Fedoseev :
--
pull_requests: +15098
pull_request: https://github.com/python/cpython/pull/15388
___
Python tracker
<https://bugs.python.org/issue27
Change by Sergey Fedoseev :
--
pull_requests: +15095
pull_request: https://github.com/python/cpython/pull/15386
___
Python tracker
<https://bugs.python.org/issue27
Change by Sergey Fedoseev :
--
pull_requests: +15094
pull_request: https://github.com/python/cpython/pull/15385
___
Python tracker
<https://bugs.python.org/issue27
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +15074
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15363
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
PyLong_As*() functions computes result for large longs like this:
size_t x, prev;
x = 0;
while (--i >= 0) {
prev = x;
x = (x << PyLong_SHIFT) | v->ob_digit[i];
if ((x >> PyLong_SHIFT) != prev) {
*overflow = sign;
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +14975
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15254
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
Argument Clinic generates `{NULL, NULL}` initializer for Py_buffer variables.
Such initializer zeroes all Py_buffer members, but as I understand only `obj`
and `buf` members are really had to be initialized. Avoiding unneeded
initialization provides tiny
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +14972
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15250
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
bytearray_getitem() adjusts negative index, though that's already done by
PySequence_GetItem().
This makes PySequence_GetItem(bytearray(1), -2) to return 0 instead of raise
IndexError.
--
components: Interpreter Core
messages: 349545
nosy: sir
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +14971
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15251
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
When compiled with default NSMALLPOSINTS, _PyLong_FromUnsignedChar() is
significantly faster than other PyLong_From*():
$ python -m perf timeit -s "from collections import deque; consume =
deque(maxlen=0).extend; b = bytes(2**20)" "consume
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +14924
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15192
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
Currently PyLong_FromSize_t() uses PyLong_FromLong() for values < PyLong_BASE.
It's suboptimal because PyLong_FromLong() needs to handle the sign. Removing
PyLong_FromLong() call and handling small ints directly in PyLong_FromSize_t()
makes it fas
Sergey Fedoseev added the comment:
`BytesIO.write()` and `BytesIO.writelines()` are independent of each other.
--
___
Python tracker
<https://bugs.python.org/issue34
Change by Sergey Fedoseev :
--
pull_requests: +13420
___
Python tracker
<https://bugs.python.org/issue31862>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
Added file: https://bugs.python.org/file48173/build_tuple_bench.py
___
Python tracker
<https://bugs.python.org/issue36030>
___
___
Python-bug
Change by Sergey Fedoseev :
Added file: https://bugs.python.org/file48174/list_as_tuple_bench.py
___
Python tracker
<https://bugs.python.org/issue36030>
___
___
Pytho
Sergey Fedoseev added the comment:
>> This optimization also can be used for BUILD_TUPLE opcode and in pickle
>> module, if it's OK to add _PyTuple_StealFromArray() function :-)
> I would like to see a micro-benchmark showing t
Sergey Fedoseev added the comment:
> Can you please convert msg336142 into a perf script?
> And then run again these benchmarks on PR 12052.
++-+--+
| Benchmark | ref | unt
Sergey Fedoseev added the comment:
This optimization also can be used for BUILD_TUPLE opcode and in pickle module,
if it's OK to add _PyTuple_StealFromArray() function :-)
--
___
Python tracker
<https://bugs.python.org/issue36
Sergey Fedoseev added the comment:
I added WIP PR with discussed micro-optimization, here are benchmark results:
$ python -m perf compare_to --min-speed 1 -G master.json tuple-untracked.json
Slower (1):
- sqlite_synth: 5.16 us +- 0.10 us -> 5.22 us +- 0.08 us: 1.01x slower (+1%)
Faster
Change by Sergey Fedoseev :
--
pull_requests: +12078
___
Python tracker
<https://bugs.python.org/issue36030>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
pull_requests: +12062
___
Python tracker
<https://bugs.python.org/issue36030>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +12008
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue36073>
___
_
New submission from Sergey Fedoseev :
It's somewhat similar to bpo-10811, but for converter function:
In [197]: import sqlite3 as sqlite
...: con = sqlite.connect(':memory:', detect_types=sqlite.PARSE_COLNAMES)
...: cur = con.cursor()
...: sqlite.converters['CURSOR_INIT
Change by Sergey Fedoseev :
--
title: str.translate() behave differently for ASCII-only and other strings ->
str.translate() behaves differently for ASCII-only and other strings
___
Python tracker
<https://bugs.python.org/issu
New submission from Sergey Fedoseev :
In [186]: from itertools import cycle
In [187]: class ContainerLike:
...: def __init__(self):
...: self.chars = cycle('12')
...: def __getitem__(self, key):
...: return next(self.chars)
...:
In [188
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +11998
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue36063>
___
_
New submission from Sergey Fedoseev :
This change produces minor speed-up:
$ python-other -m perf timeit -s "divmod_ = divmod" "divmod_(1, 1)"
--duplicate=1000 --compare-to=../cpython-master/venv/bin/python
python: . 64.6 ns +-
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +11993
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue36062>
___
_
New submission from Sergey Fedoseev :
list_slice() is used by PyList_GetSlice(), list_subscript() and for list
copying. In list_subscript() slice indices are already normalized with
PySlice_AdjustIndices(), so slice normalization currently performed in
list_slice() is only needed
Sergey Fedoseev added the comment:
Here's benchmarks for PyTuple_FromArray() PR:
$ python -m perf timeit -s "l = [None]*0; tuple_ = tuple" "tuple_(l)"
--duplicate=100 --compare-to=../cpython-master/venv/bin/python
/home/sergey/tmp/cpython-master/venv/bin/python: ..
Change by Sergey Fedoseev :
--
pull_requests: +11979
___
Python tracker
<https://bugs.python.org/issue36030>
___
___
Python-bugs-list mailing list
Unsubscribe:
Sergey Fedoseev added the comment:
Does this look like more real world example?
Before:
$ python -m perf timeit -s "t = (1, 2, 3)" "(*t, 4, 5)"
.
Mean +- std dev: 95.7 ns +- 2.3 ns
After:
$ python -m perf timeit -s "t = (1, 2, 3)" "(*t
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +11953
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue36031>
___
_
New submission from Sergey Fedoseev :
There are several cases in CPython sources when PyList_AsTuple() is used with
just created list and immediately after that this list is Py_DECREFed. This
operation can be performed more effectively since refcount of items is not
changed. For example
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +11952
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue36030>
___
_
New submission from Sergey Fedoseev :
PyTuple_New() fills items array with NULLs to make usage of Py_DECREF() safe
even when array is not fully filled with real items.
There are multiple cases when this initialization step can be avoided to
improve performance. For example it gives such speed
Change by Sergey Fedoseev :
--
pull_requests: +10490
___
Python tracker
<https://bugs.python.org/issue33234>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue34995>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue23460>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue28397>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue34898>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue32788>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue21165>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/issue34751>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +8859
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue34749>
___
___
Py
New submission from Sergey Fedoseev :
I reworked implementation of binascii.a2b_base64() and there is significant
speedup:
$ python -m perf timeit --compare-to ~/tmp/cpython-master-venv/bin/python -s
"from binascii import b2a_base64, a2b_base64; s = b2a_base64(1000 * b'b')"
&qu
Sergey Fedoseev added the comment:
> Now you rely on PyMem_Malloc() to detect the overflow.
Now overflow is not possible or am I missing something?
--
nosy: +sir-sigurd
___
Python tracker
<https://bugs.python.org/iss
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +8541
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue34599>
___
___
Py
New submission from Sergey Fedoseev :
Currently _Py_bytes_capitalize() checks case before using conversion table,
performance can be improved by using conversion table unconditionally.
Benchmarks:
$ python -m perf timeit --compare-to ~/tmp/cpython-master-venv/bin/python -s "b
Sergey Fedoseev added the comment:
Other iterators either don't support pickling or aren't mutated during pickling:
In [10]: it = iter({1})
In [11]: pickle.dumps(it)
Out[11]:
b'\x80\x04\x95\x1e\x00\x00\x00\x00\x00\x00\x00\x8c\x08builtins\x94\x8c\x04iter\x94\x93\x94]\x94K\x01a\x85\x94R\x94
Change by Sergey Fedoseev :
--
pull_requests: +8519
___
Python tracker
<https://bugs.python.org/issue1621>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +8513
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue34574>
___
___
Py
New submission from Sergey Fedoseev :
In [1]: from collections import OrderedDict
In [2]: od = OrderedDict.fromkeys(range(10))
In [3]: it = iter(od)
In [4]: it.__reduce__()
Out[4]: (, ([0, 1, 2, 3, 4, 5, 6, 7, 8, 9],))
In [5]: list(it)
Out[5]: []
--
messages: 324551
nosy: sir
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +8512
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue34573>
___
___
Py
New submission from Sergey Fedoseev :
Currently __reduce__() implementation of set and dict iterators saves items to
list using next/PyList_Append(), it could be simplified by using
PySequence_List().
--
components: Interpreter Core
messages: 324550
nosy: sir-sigurd
priority: normal
Change by Sergey Fedoseev :
--
keywords: +patch
pull_requests: +8374
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue34488>
___
___
Py
New submission from Sergey Fedoseev :
Currently BytesIO.writelines() uses BytesIO.write() which returns number of
written bytes as Python ints, but these ints are not used by
BytesIO.writelines(), so avoiding creation of these ints can improve
performance of BytesIO.writelines() especially
Change by Sergey Fedoseev :
--
pull_requests: +8262
___
Python tracker
<https://bugs.python.org/issue34395>
___
___
Python-bugs-list mailing list
Unsubscribe:
1 - 100 of 136 matches
Mail list logo
