Change by Yury Selivanov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
type: -> enhancement
___
Python tracker
<https://bugs.python
Change by Yury Selivanov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
New changeset f13f12d8daa587b5fcc66fe3ed1090a5dadab289 by Yury Selivanov in
branch 'master':
bpo-32630: Use contextvars in decimal (GH-5278)
https://github.com/python/cpython/commit/f13f12d8daa587b5fcc66fe3ed1090
Yury Selivanov added the comment:
Thank you, Stefan. I've updated the PR with an asyncio+decimal test and run
tests in refleak mode to make sure there's no regression there.
If during the beta/rc period we see that contextvars isn't stable enough or
something I'll rever
Yury Selivanov added the comment:
New changeset bc4123b0b380edda774b8bff2fa1bcc96453b440 by Yury Selivanov in
branch 'master':
bpo-32436: Use PyThreadState_GET() in all hot paths (GH-5363)
https://github.com/python/cpython/commit/bc4123b0b380edda774b8bff2fa1bc
Change by Yury Selivanov :
--
pull_requests: +5207
___
Python tracker
<https://bugs.python.org/issue32436>
___
___
Python-bugs-list mailing list
Unsubscribe:
Yury Selivanov added the comment:
New changeset 226e50049da43097d89e9e99f9c55f212d1f7bd5 by Yury Selivanov in
branch 'master':
bpo-32436: Make PyContextVar_Get a little bit faster (#5350)
https://github.com/python/cpython/commit/226e50049da43097d89e9e99f9c55f
Yury Selivanov added the comment:
Sorry Stefan, I never wanted this to look like "I'm pushing this without
listening to Stefan". I apologize if it looked that way.
I ran bm_telco on my machine before submitting the PR, and I indeed did not see
any performance impact. I
Change by Yury Selivanov :
--
pull_requests: +5197
___
Python tracker
<https://bugs.python.org/issue32436>
___
___
Python-bugs-list mailing list
Unsubscribe:
Yury Selivanov added the comment:
> However, I could not find any tests for the added feature (safe
> use with async) though. We would be adding a new feature without
> tests.
This is no problem, I can add a few async/await tests.
> I'm getting a large slowdown:
> ./pyt
Change by Yury Selivanov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
New changeset 43c47fe09640c579462978ec16f81295f5857cde by Yury Selivanov in
branch 'master':
bpo-32670: Enforce PEP 479. (#5327)
https://github.com/python/cpython/commit/43c47fe09640c579462978ec16f812
Yury Selivanov added the comment:
Closing this one now. Thanks Serhiy and Inada-san!
--
nosy: +yselivanov
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
I think these failures were on the previous version of the test; i've fixed it
last night and it shouldn't timeout this way anymore.
--
___
Python tracker
<https://bugs.python.o
Yury Selivanov added the comment:
> OK. I hope both of you keep an eye on this and actively try to test and
break it!
That's the plan!
--
___
Python tracker
<https://bugs.python.org
Change by Yury Selivanov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
Nathaniel's latest PR looks good and fixes the crash.
FWIW losing some warnings during interpreter shutdown isn't exactly a new
problem, therefore I think there's nothing wrong with the new API here. For
instance, I've seen asyncio&
Yury Selivanov added the comment:
New changeset dba976b8a28d6e5daa66ef31a6a7c694a9193f6a by Yury Selivanov
(Nathaniel J. Smith) in branch 'master':
bpo-32591: fix abort in _PyErr_WarnUnawaitedCoroutine during shutdown (#5337)
https://github.com/python/cpyt
Yury Selivanov added the comment:
New changeset 4112c5b97d9c1c7b034653d0e017ffa894a45c74 by Yury Selivanov in
branch 'master':
bpo-32662: Try making test_asyncio.test_server more reliable (#5338)
https://github.com/python/cpython/commit/4112c5b97d9c1c7b034653d0e017ff
Change by Yury Selivanov :
--
pull_requests: +5185
___
Python tracker
<https://bugs.python.org/issue32662>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Yury Selivanov :
--
resolution: fixed ->
status: closed -> open
___
Python tracker
<https://bugs.python.org/issue32591>
___
___
Python-bugs-list
Yury Selivanov added the comment:
The following snippet crashes with SIGABRT:
import asyncio
async def f(): pass
asyncio.gather(f())
llvm tells that the crash is in _PyGen_Finalize/_PyErr_WarnUnawaitedCoroutine
--
___
Python tracker
Change by Yury Selivanov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
New changeset c47dacb69054f6fe1c2465df585985430f7fe366 by Yury Selivanov (José
Melero Fernández) in branch 'master':
bpo-32574: Fix leaks in asyncio.Queue.put() and .get() (#5208)
https://github.com/python/cpyt
Change by Yury Selivanov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
New changeset c9070d03f5169ad6e171e641b7fa8feab18bf229 by Yury Selivanov in
branch 'master':
bpo-32662: Implement Server.start_serving() and Server.serve_forever() (#5312)
https://github.com/python/cpython/commit/c9070d03f5169ad6e171e641b7fa8f
Yury Selivanov added the comment:
> Sure, and *I* am the one running the extended decimal test suite as we speak,
> not Guido.
Thank you.
> You are playing power games here, and you did that from the start by choosing
> the nosy list.
Please. I thought it was pretty much dec
Yury Selivanov added the comment:
Stefan, I do think that this is a release blocker. We want to get this change
as early as possible to ensure that it's well tested.
AFAIK Guido also wants decimal to be updated and well supported in async/await
Change by Yury Selivanov :
--
nosy: +ned.deily
priority: normal -> release blocker
___
Python tracker
<https://bugs.python.org/issue32630>
___
___
Python-
Change by Yury Selivanov :
--
keywords: +patch
pull_requests: +5172
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue32670>
___
___
Py
New submission from Yury Selivanov :
As per PEP 479, in Python 3.7 it should be a RuntimeError when a StopIteration
exception is raised manually in a generator, a coroutine, or an async generator.
--
assignee: yselivanov
components: Interpreter Core
messages: 310712
nosy: gvanrossum
Yury Selivanov added the comment:
Thank you Alexander and Ilya!
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
New changeset 1aa094f74039cd20fdc7df56c68f6848c18ce4dd by Yury Selivanov (Ilya
Kulakov) in branch 'master':
bpo-29302: Implement contextlib.AsyncExitStack. (#4790)
https://github.com/python/cpython/commit/1aa094f74039cd20fdc7df56c68f68
Yury Selivanov added the comment:
I pushed a fix (already in the master branch) and rebased the patch once again.
I expect it to work now :)
--
___
Python tracker
<https://bugs.python.org/issue32
Yury Selivanov added the comment:
New changeset 6ab62920c87930dedc31fe633ecda3e51d3d7503 by Yury Selivanov in
branch 'master':
bpo-32436: Fix a refleak; var GC tracking; a GCC warning (#5326)
https://github.com/python/cpython/commit/6ab62920c87930dedc31fe633ecda3
Yury Selivanov added the comment:
I think I found what cause this, but I have no idea why it has surfaced only
now :)
https://github.com/python/cpython/pull/5326/files
(see the added PyType_IS_GC(Py_TYPE(name)) check)
I'll merge that PR and rebase the decimal patch
Yury Selivanov added the comment:
(Just in case I rebased my patch onto the latest master)
--
___
Python tracker
<https://bugs.python.org/issue32630>
___
___
Change by Yury Selivanov :
--
pull_requests: +5171
___
Python tracker
<https://bugs.python.org/issue32436>
___
___
Python-bugs-list mailing list
Unsubscribe:
Yury Selivanov added the comment:
> I realize that you had to fight massive mailing list distractions
> during the PEP discussions, but this is very close to the beta ...
Oh thanks, but I see no reason for you to be condescending here.
I cannot reproduce this on Mac OS / Linux. Are yo
Change by Yury Selivanov :
--
keywords: +patch
pull_requests: +5158
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue32662>
___
___
Py
New submission from Yury Selivanov :
As discussed, we want to make Server objects more usable in async/await code
and more compatible with asyncio.run.
This is also needed to handle a use case when two or more servers are created
and need to start listening at the same time.
We propose to
Yury Selivanov added the comment:
Stefan, it would be great to have this committed before 3.7 feature freeze.
The change is pretty straightforward -- we replaced threading.local() with a
contextvar, which should be a backwards compatible change
Yury Selivanov added the comment:
This can probably be closed now. Thanks Victor and Nathaniel!
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
New changeset 789e359f51d2b27bea01b8c6c3bf090aaedf8839 by Yury Selivanov
(Victor Stinner) in branch 'master':
bpo-32636: Fix two bugs in test_asyncio (#5302)
https://github.com/python/cpython/commit/789e359f51d2b27bea01b8c6c3bf09
Yury Selivanov added the comment:
New changeset fb5a7ad421ac20c49218ee4b86fb0d85ca4cd664 by Yury Selivanov
(Nathaniel J. Smith) in branch 'master':
bpo-32636: Fix @asyncio.coroutine debug mode bug exposed by gh-5250 (#5291)
https://github.com/python/cpyt
Yury Selivanov added the comment:
Can you check if this happens on 3.6? 3.5 is in a security-fix mode only, so
even if this is a bug it would take us a long time to fix it.
--
___
Python tracker
<https://bugs.python.org/issue32
Yury Selivanov added the comment:
So let's retarget this to 3.8 as it's too close to 3.7 feature freeze to get
seriously considered/merged.
--
versions: +Python 3.8 -Python 3.7
___
Python tracker
<https://bugs.python.o
Change by Yury Selivanov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
New changeset 22feeb88b473b288950cdb2f6c5d28692274b5f9 by Yury Selivanov in
branch 'master':
bpo-32643: Drop support for a few private Task and Future APIs. (#5293)
https://github.com/python/cpython/commit/22feeb88b473b288950cdb2f6c5d28
Yury Selivanov added the comment:
> Nathaniel: do you want to include these fixes in your PR as well?
Victor, can you submit a new PR for this? Let's merge this right away.
--
___
Python tracker
<https://bugs.python.org
Yury Selivanov added the comment:
I now have a Win7 VM, I'll take a look at this later.
--
___
Python tracker
<https://bugs.python.org/issue32645>
___
___
Change by Yury Selivanov :
--
keywords: +patch
pull_requests: +5140
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue32643>
___
___
Py
New submission from Yury Selivanov :
I propose to drop support for overloading Task._step, Task._wakeup, and
Future._schedule_callbacks.
This makes the implementation easier for us to maintain and optimize. It also
makes it easier for alternative asyncio event loops and interops with other
Change by Yury Selivanov :
--
nosy: +gvanrossum
___
Python tracker
<https://bugs.python.org/issue30491>
___
___
Python-bugs-list mailing list
Unsubscribe:
Yury Selivanov added the comment:
Guido,
This is another feature for native coroutines that Nathaniel proposes for 3.7.
Here's a summary (this issue has too many messages):
1. It adds a new mechanism to detect un-awaited coroutines (i.e. when a user
forgets to use await).
2. To enabl
Yury Selivanov added the comment:
New changeset b7a80d543e1e94475ab9c8214f7a9eab4e63c9ab by Yury Selivanov in
branch 'master':
bpo-32436: Don't use native popcount() (also fixes bpo-32641) (#5292)
https://github.com/python/cpython/commit/b7a80d543e1e94475ab9c8214
Yury Selivanov added the comment:
I've pushed a commit that should fix the buildbot. Please reopen if it doesn't.
--
___
Python tracker
<https://bugs.python.o
Yury Selivanov added the comment:
New changeset b7a80d543e1e94475ab9c8214f7a9eab4e63c9ab by Yury Selivanov in
branch 'master':
bpo-32436: Don't use native popcount() (also fixes bpo-32641) (#5292)
https://github.com/python/cpython/commit/b7a80d543e1e94475ab9c8214
Yury Selivanov added the comment:
In an off-list conversation with David, it was confirmed that the buildbot in
question runs on an old hardware without SSE4.2 support.
I decided to simply stop using native popcount instructions as there's no
detectable performance difference when using
Change by Yury Selivanov :
--
keywords: +patch
pull_requests: +5139
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue32641>
___
___
Py
Change by Yury Selivanov :
--
pull_requests: +5138
stage: commit review -> patch review
___
Python tracker
<https://bugs.python.org/issue32436>
___
___
Python-
Yury Selivanov added the comment:
No, it's this specific commit. I can double check later, but I'm pretty sure
about that.
--
___
Python tracker
<https://bugs.python.o
Yury Selivanov added the comment:
asyncio.coroutine sets the co_flags bit CO_ITERABLE_COROUTINE for generator
functions' code objects.
With that bit flag, Python allows to 'yield from' native coroutines.
CoroWrapper.send() -> wrapped_generator.send() -> Y
Yury Selivanov added the comment:
I suspect this is a compiler bug. I can't reproduce it on my windows 7 virtual
machine and on AppVeyor.
It's not related to HAMT, btw, as there is a multitude of HAMT-specific tests
that all pass. The crash is specifically in context.c, and the
Yury Selivanov added the comment:
No, these are trivial uint32_t/Py_SIZE: `for (uint32_t i = val_idx + 1; i <
Py_SIZE(o); i++)`; it's something else. I suspect popcount instruction...
--
___
Python tracker
<https://bugs.python.org
Yury Selivanov added the comment:
It's not a regression, the newly added code doesn't work on some Windows
buildbot... looking at it.
--
___
Python tracker
<https://bugs.python.o
Yury Selivanov added the comment:
> Oh, I didn't know. The current workflow is not ideal to avoid duplicated
> issues.
If I fix something that's been reported there I usually update the coverity
issue directly. But yeah...
--
__
Yury Selivanov added the comment:
Victor, it's been already fixed/tracked here:
https://github.com/python/cpython/pull/5286
(I also receive coverity reports :)
--
resolution: -> out of date
stage: -> resolved
status: open -> closed
type
Yury Selivanov added the comment:
New changeset 0bad4d63c654d93e1f32ff35026405a3987db5ca by Yury Selivanov in
branch 'master':
bpo-32436: Fix potential NULL dereference (#5286)
https://github.com/python/cpython/commit/0bad4d63c654d93e1f32ff35026405
Change by Yury Selivanov :
--
stage: patch review -> commit review
___
Python tracker
<https://bugs.python.org/issue32436>
___
___
Python-bugs-list mai
Change by Yury Selivanov :
--
pull_requests: +5132
___
Python tracker
<https://bugs.python.org/issue32436>
___
___
Python-bugs-list mailing list
Unsubscribe:
Yury Selivanov added the comment:
Andrew, test_cancel2 test fails 1 out of 10 times on my machine (log below).
Could you please take a look?
Also, can you rename sendfile tests to 'test_sock_sendfile_*'?
{pydev} ~/d/p/cpython (master %) » ./python.exe -m test test_asy
Yury Selivanov added the comment:
New changeset 9d411c119fdd8e42209ec16be27686a843507f18 by Yury Selivanov in
branch 'master':
bpo-32296: Make get_running_loop() another 4-5x faster (#5277)
https://github.com/python/cpython/commit/9d411c119fdd8e42209ec16be27686
Yury Selivanov added the comment:
However I ran the whole asyncio test suite in reftracking mode on my machine
(latest master) and it passed. So I guess this issue is indeed closed.
--
___
Python tracker
<https://bugs.python.org/issue32
Yury Selivanov added the comment:
I'd like to see benchmarks of coroutine creation time and awaits with and
without the proposed patch.
--
___
Python tracker
<https://bugs.python.org/is
Yury Selivanov added the comment:
I don't think that 6934831e43d66222a626403dd775429d1c8963f3 can address the
leak in any way.
--
___
Python tracker
<https://bugs.python.org/is
Yury Selivanov added the comment:
Nathaniel, this is is a regression of fc2f407829d9817ddacccae6944dd0879cfaca24
-- bpo-32591: Add native coroutine origin tracking.
Please take a look.
--
nosy: +njs
___
Python tracker
<https://bugs.python.
Yury Selivanov added the comment:
New changeset 83c8675edb4fe278c5d930f7865977a5d3c7168a by Yury Selivanov in
branch 'master':
bpo-32436: Remove a redundant assert (#5275)
https://github.com/python/cpython/commit/83c8675edb4fe278c5d930f7865977
Change by Yury Selivanov :
--
keywords: +patch
pull_requests: +5122
___
Python tracker
<https://bugs.python.org/issue32630>
___
___
Python-bugs-list mailin
New submission from Yury Selivanov :
PEP 567 allows decimal to be safely used in async/await code.
I couldn't observe any performance impact by the proposed PR. The PR doesn't
modify decimal context behaviour: instead of using a thread-local storage it
now uses a contex
Change by Yury Selivanov :
--
pull_requests: +5121
___
Python tracker
<https://bugs.python.org/issue32296>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Yury Selivanov :
--
pull_requests: +5120
___
Python tracker
<https://bugs.python.org/issue32436>
___
___
Python-bugs-list mailing list
Unsubscribe:
Yury Selivanov added the comment:
This one: https://github.com/python/cpython/pull/5273? I've left a comment with
some questions there. Are there any other urgent PRs?
The main PEP 567 PR has been merged already, btw.
--
___
Python tracker
&
Yury Selivanov added the comment:
I'll try to find some time after beta-1. Right now have a few other issues
that I have to finish (and also create a PR to make decimal use PEP 567).
--
___
Python tracker
<https://bugs.python.org/is
Yury Selivanov added the comment:
I'll keep the issue open -- a documentation for the new APIs will be merged in
a separate PR in a couple of months.
--
___
Python tracker
<https://bugs.python.org/is
New submission from Yury Selivanov :
New changeset f23746a934177c48eff754411aba54c31d6be2f0 by Yury Selivanov in
branch 'master':
bpo-32436: Implement PEP 567 (#5027)
https://github.com/python/cpython/commit/f23746a934177c48eff754411aba54
Yury Selivanov added the comment:
They have been documented as coroutines and were returning futures since Python
3.4. I believe we can keep the status quo especially if only one user
complained about it during the last 5 years.
--
___
Python
Yury Selivanov added the comment:
Since we made them coroutines in 3.7, I'd prefer to keep the 3.6 documentation
as is (so that people don't write code that expects them to be not coroutines
in 3.6).
Please let's keep the cur
Change by Yury Selivanov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
New changeset b0a7a037b8fde56b62f886d5188bced7776777b4 by Yury Selivanov in
branch 'master':
bpo-31179: Make dict.copy() up to 5.5 times faster. (#3067)
https://github.com/python/cpython/commit/b0a7a037b8fde56b62f886d5188bce
Yury Selivanov added the comment:
> Andrew, shouldn't we fix 3.6 documentation? sock_* methods still documented
> as coroutines.
We only fixed it in 3.7 and the doc was updated. 3.6 stays as is.
--
___
Python tracker
<https://b
Yury Selivanov added the comment:
Victor: done; https://bugs.python.org/issue32623
--
___
Python tracker
<https://bugs.python.org/issue31179>
___
___
Python-bug
New submission from Yury Selivanov :
We should investigate whether we want dicts to compact themselves on del/pop
operations. Currently we have to rely on workarounds to have compactable
dict.copy (see issue 31179) etc.
--
components: Interpreter Core
messages: 310427
nosy
Yury Selivanov added the comment:
+1 to what Andres said.
--
nosy: +yselivanov
resolution: -> not a bug
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Yury Selivanov added the comment:
New changeset 3510334361d6d39aad89e4e0d9bccd0695668583 by Yury Selivanov in
branch 'master':
bpo-32591: Fix PyExc_WarnFormat call (follow-up commit) (#5263)
https://github.com/python/cpython/commit/3510334361d6d39aad89e4e0d9bccd
Change by Yury Selivanov :
--
pull_requests: +5109
___
Python tracker
<https://bugs.python.org/issue32591>
___
___
Python-bugs-list mailing list
Unsubscribe:
Yury Selivanov added the comment:
New changeset a4afcdfa55ddffa4b9ae3b0cf101628c7bff4102 by Yury Selivanov in
branch 'master':
bpo-32314: Fix asyncio.run() to cancel runinng tasks on shutdown (#5262)
https://github.com/python/cpython/commit/a4afcdfa55ddffa4b9ae3b0cf10162
Yury Selivanov added the comment:
I've pushed a new version of the patch that I intend to merge tomorrow.
The last version has only one minor change: it uses fast-path for "slightly"
non-compact dicts too (dicts don't use *at most* 3 entries). This protects us
from patho
Change by Yury Selivanov :
--
pull_requests: +5107
___
Python tracker
<https://bugs.python.org/issue32314>
___
___
Python-bugs-list mailing list
Unsubscribe:
Yury Selivanov added the comment:
I agree, returning done tasks is pretty useless.
This is a new function so let's fix its behavour. We need to:
1. Document this difference between new asyncio.all_tasks() and now deprecated
Task.all_tasks().
2. Make sure that Task.all_tasks() works i
901 - 1000 of 3098 matches
Mail list logo