Changes by Andrew Svetlov :
--
nosy: +asvetlov
___
Python tracker
<http://bugs.python.org/issue3405>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
Is there actual?
--
nosy: +asvetlov
versions: +Python 3.3 -Python 3.2
___
Python tracker
<http://bugs.python.org/issue1
Andrew Svetlov added the comment:
I've updated the patch.
If nobody object I'll commit it soon.
--
nosy: +asvetlov
versions: +Python 3.3 -Python 2.7, Python 3.1, Python 3.2
Added file: http://bugs.python.org/file25110/issue3033.diff
Andrew Svetlov added the comment:
Thank you, Ned.
--
___
Python tracker
<http://bugs.python.org/issue1584>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
Brian, please don't forget to cleanup subprocess code when you will be ready to.
--
assignee: -> brian.curtin
___
Python tracker
<http://bugs.python.org
Andrew Svetlov added the comment:
I think we have to reject this issue.
Adding generic validation make sense, but adding just permanent check for
non-empty string is wrong.
--
nosy: +asvetlov
___
Python tracker
<http://bugs.python.org/issue12
Changes by Andrew Svetlov :
--
nosy: +asvetlov
___
Python tracker
<http://bugs.python.org/issue6181>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
Tests passed, looks good. Code is also looks ok.
Documentation is out-of-date. Patch cannot be applied to struct.rst from
default branch.
Compilation generates warning messages on gcc 4.6.1:
/home/andrew/projects/py3k/Modules/_struct.c: In function
Andrew Svetlov added the comment:
I used half-float in GPU programming and from my perspective it was just
native. There are no half-float in C, right. But there are half-floats used in
NVIDIA libraries for example and I like to think used format is native and
platform-depended in general
Andrew Svetlov added the comment:
Antoine, agree with you after explanation.
--
___
Python tracker
<http://bugs.python.org/issue11734>
___
___
Python-bugs-list m
Andrew Svetlov added the comment:
Closing as wan't fix.
--
assignee: kbk -> asvetlov
resolution: -> wont fix
stage: test needed -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.pyt
Andrew Svetlov added the comment:
The patch looks good.
Can you describe manual steps to reproduce the issue?
--
assignee: -> asvetlov
nosy: +asvetlov
___
Python tracker
<http://bugs.python.org/issu
Andrew Svetlov added the comment:
Closing as fixed
--
assignee: -> asvetlov
resolution: -> fixed
stage: patch review -> committed/rejected
___
Python tracker
<http://bugs.python.o
Andrew Svetlov added the comment:
Thanks to Guilherme Polo.
--
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue3033>
___
___
Python-
Changes by Andrew Svetlov :
--
nosy: +asvetlov
___
Python tracker
<http://bugs.python.org/issue6225>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Andrew Svetlov :
--
nosy: +asvetlov
___
Python tracker
<http://bugs.python.org/issue6167>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Andrew Svetlov :
--
nosy: +asvetlov
___
Python tracker
<http://bugs.python.org/issue9141>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
Fixed. Thanks.
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Andrew Svetlov added the comment:
Closing as won't fix.
--
assignee: kbk -> asvetlov
nosy: +asvetlov
resolution: -> wont fix
stage: test needed -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.pyt
Andrew Svetlov added the comment:
Closing as not reproducible for 3.3.
The bug is too minor to worry about 2.7 and 3.2 even if it present (all works
for me by the way).
Please reopen if you will get described issue.
--
assignee: -> asvetlov
nosy: +asvetlov
resolution: ->
Andrew Svetlov added the comment:
What's about test which pass bytes to Popen?
Should it be deprecated and should Popen accept only unicode strings only — I
mean `str` type?
As I know the trend of py3k to get rid of bytes in filesystem
Andrew Svetlov added the comment:
Thanks to all.
Bruce Frederiksen, you are mentioned in Misc/ACK
Tal Einat, if you want to make a patch which will use `execfile` instead of
current approach — you are welcome. Please file new issue.
--
assignee: -> asvetlov
resolution: ->
Changes by Andrew Svetlov :
--
assignee: gpolo -> asvetlov
___
Python tracker
<http://bugs.python.org/issue7057>
___
___
Python-bugs-list mailing list
Unsubscri
Andrew Svetlov added the comment:
If fylesystem doesn't support unicode (only Windows directly does as I know)
str filename should to be converted by `sys.getfilesystemencoding()`. `os.exec`
family already does it as well as other fs functions — but they are supports
bytes also.
Andrew Svetlov added the comment:
Tkinter (as Tcl itself) has no support of non-BMP characters in any form.
It looks like support of UTF-16 without surrogates.
I like to implement codec for that which will process different error modes
(strict, replace, ignore etc) as well as others codecs
Changes by Andrew Svetlov :
--
resolution: -> rejected
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue12723>
___
___
Python-bugs-
Andrew Svetlov added the comment:
I guess IdleConf should to have flag like 'writable'.
If user environment points to invalid location (or there are no write access)
this flag should be set.
It flag can affect IDLE configuration dialog: user should be notified what him
changes w
Andrew Svetlov added the comment:
The way is named 'codec'.
--
___
Python tracker
<http://bugs.python.org/issue14304>
___
___
Python-bugs-list mailing
Change by Andrew Svetlov :
--
pull_requests: +13301
___
Python tracker
<https://bugs.python.org/issue32972>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
https://github.com/python/cpython/pull/13386 is a new attempt to solve the
feature request
--
___
Python tracker
<https://bugs.python.org/issue32
Change by Andrew Svetlov :
--
keywords: +patch
pull_requests: +13349
___
Python tracker
<https://bugs.python.org/issue29883>
___
___
Python-bugs-list mailin
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.7
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
The correct approach is using Protocol.pause_writing() /
Protocol.resume_writing() callbacks.
No trivial, though.
See `StreamWriter.drain()` for example of implementation.
--
___
Python tracker
<ht
Andrew Svetlov added the comment:
I'm sorry but I cannot tell why you are using `pause_writing` incorrectly
without looking on the code.
--
___
Python tracker
<https://bugs.python.org/is
Andrew Svetlov added the comment:
No. It doesn't work this way.
pause_writing is a protocol callback called from transport when the internal
buffer is full.
In reaction to this callback the producer should stop calling transport.write()
and resume writing after getting `resume_wr
Andrew Svetlov added the comment:
No. pause_writing/resume_writing are protocol callbacks called by transport.
User code should respond to these callbacks by stopping sending data to
transport (transport.write()).
The logic is a little complicated but it is ok for very low-level asyncio API
Andrew Svetlov added the comment:
Please re-read my previous comment.
If you use asyncio incorrectly -- yes, you can run out of memory.
The proper usage of pause_readind()/resume_reading() resolves the issue.
--
___
Python tracker
<ht
Andrew Svetlov added the comment:
Thank you very much for raising the question.
@patch(...) creates _patch class instance.
For decoration _patch.__call__ is used.
def __call__(self, func):
if isinstance(func, type):
return self.decorate_class(func)
return
Andrew Svetlov added the comment:
Great!
--
___
Python tracker
<https://bugs.python.org/issue34616>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
Please keep in mind: not only asyncio can be used to execute async/await code.
For example, trio has completely different implementation but utilizes async
functions as well.
Probably we need to customize it, maybe by inheriting AsyncioDocTestRunner from
New submission from Andrew Svetlov :
After merging https://github.com/python/cpython/pull/9296 asyncio test suite
prints a lot of warnings like
Exception ignored in:
Traceback (most recent call last):
File "/home/andrew/projects/cpython/Lib/warnings.py", li
Change by Andrew Svetlov :
--
title: Fix asyncio mock wranings -> Fix asyncio mock warnings
___
Python tracker
<https://bugs.python.org/issue37015>
___
___
Py
Andrew Svetlov added the comment:
Agree with Nathaniel.
There is no need to rush now.
--
___
Python tracker
<https://bugs.python.org/issue37006>
___
___
Pytho
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
versions: +Python 3.7
___
Python tracker
<https://bugs.python.org/issue37008>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrew Svetlov added the comment:
xtreak thanks for investigation.
I have no time to work on it now, sorry.
Maybe the next week.
Anyway, the problem can be fixed even in python beta stage if we don't do it
earlier.
--
___
Python tracker
&
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue36802>
___
___
Python-bugs-list
New submission from Andrew Svetlov :
Currently asyncio uses `loop.call_exception_handler()` for logging *fatal*
exceptions from underlying sockets before calling protocol.connection_lost().
There is a list of exceptions that are not logged: BrokenPipeError,
ConnectionResetError
Andrew Svetlov added the comment:
Update.
Currently, exceptions from the ignore list are still logged in debug mode. The
proposal doesn't change this behavior.
--
___
Python tracker
<https://bugs.python.org/is
Change by Andrew Svetlov :
--
keywords: +patch
pull_requests: +13460
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue37035>
___
___
Py
Andrew Svetlov added the comment:
Thanks for the report.
If the problem is in asyncio.wait_for() function the real network code can be
stripped for the leakage example and replaced with `await asyncio.sleep()`.
Would you try to boil down the snippet by converting it into a code that I can
Andrew Svetlov added the comment:
Nice, thanks!
--
___
Python tracker
<https://bugs.python.org/issue37042>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Pull Request is welcome!
--
___
Python tracker
<https://bugs.python.org/issue36686>
___
___
Python-bugs-list mailing list
Unsub
Andrew Svetlov added the comment:
Fixed by issue37035
--
resolution: -> duplicate
stage: patch review -> resolved
status: open -> closed
superseder: -> Don't log OSError exceptions in asyncio transports
___
Pytho
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
pull_requests: +13501
pull_request: https://github.com/python/cpython/pull/13594
___
Python tracker
<https://bugs.python.org/issue37
Andrew Svetlov added the comment:
New changeset a79b6c578fcd2ea8be29440fdd8a998e5527200f by Andrew Svetlov in
branch '3.7':
[3.7] bpo-37035: Don't log OSError (GH-13548) (#13594)
https://github.com/python/cpython/commit/a79b6c578fcd2ea8be29440fd
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Fixed by #36889
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
superseder: -> Merge StreamWriter and StreamReader into just asyncio.Stream
___
Python tra
Andrew Svetlov added the comment:
Fixed by #36889
--
resolution: -> fixed
stage: -> resolved
status: open -> closed
superseder: -> Merge StreamWriter and StreamReader into just asyncio.Stream
___
Python tracker
<https://
Andrew Svetlov added the comment:
The suggestion is using streams as a high-level API.
I see no real proposals in this issue.
select()/read()/write() is not suitable interface, sorry.
Closing, feel free to create a new issue if you come up with a concrete
proposal
New submission from Andrew Svetlov :
Done by #36889
--
resolution: -> fixed
stage: -> resolved
status: open -> closed
superseder: -> Merge StreamWriter and StreamReader into just asyncio.Stream
___
Python tracker
<https://
Andrew Svetlov added the comment:
FlowControlMixing is a private API class.
The class is not intended for the usage outside of asyncio.
If you found it useful for you please feel free to copy-paste the class from
asyncio sources into your project.
The reason is: we want to keep freedom for
Andrew Svetlov added the comment:
#36889 deprecates using FlowXontrolMixin outside of asyncio
--
___
Python tracker
<https://bugs.python.org/issue34993>
___
___
Andrew Svetlov added the comment:
Brett please elaborate.
Do you want to incorporate async_lru library into CPython Core?
--
___
Python tracker
<https://bugs.python.org/issue35
Change by Andrew Svetlov :
--
resolution: -> not a bug
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
asyncio uses bare concurrent.futures.ThreadPoolExecutor.
Tha question is: should asyncio reduce the number of threads in the pool or
concurrent.futures should change the default value?
--
___
Python tracker
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
All we need is `program = os.fspath(program)` in base_events.py
subprocess_exec() method.
Is anybody volunteered to make a patch (with test and docs update, sure)?
--
___
Python tracker
<https://bugs.python.
Change by Andrew Svetlov :
--
versions: +Python 3.8 -Python 3.7
___
Python tracker
<https://bugs.python.org/issue35246>
___
___
Python-bugs-list mailin
Andrew Svetlov added the comment:
I'm ok with changing the default threads number limit.
Not sure about numbers.
If you want to limit to 16-20 that may be ok but `cpu_count + 4` doesn't work
in this case. On cloud servers, I see 128 or even more cores very often. 160+4
is not tha
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
> how about min(32, cpu_count+4)?
I think it produces reasonable good numbers for any CPU count.
Do you have time for preparing a pull request?
--
___
Python tracker
<https://bugs.python.org/issu
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
You are welcome, Victor!
Thank you very much, Adam!
--
___
Python tracker
<https://bugs.python.org/issue29883>
___
___
Pytho
Andrew Svetlov added the comment:
Aha, I see. You are right.
Let's drop checks for cmd/program/args types but left to subprocess do these
checks.
This approach avoiding complex checking/converting logic in asyncio itself and
reduces code duplic
Change by Andrew Svetlov :
--
resolution: -> fixed
versions: -Python 3.5, Python 3.6, Python 3.7, Python 3.9
___
Python tracker
<https://bugs.python.org/issu
Andrew Svetlov added the comment:
>From my understanding, the correct code should close all transports and wait
>for their connection_lost() callbacks before closing the loop.
--
___
Python tracker
<https://bugs.python.org/i
Change by Andrew Svetlov :
--
keywords: +patch
pull_requests: +13530
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/13630
___
Python tracker
<https://bugs.python.org/issu
Andrew Svetlov added the comment:
*Passing* loop to gather should be deprecated.
*Using* loop by *internal logic* is pretty fine, we do it in asyncio everywhere.
Yuri, I think we should deprecate explicit loop everywhere in non-deprecated
asyncio API by Python 3.8.
We can do it even in
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Let's postpone to 3.9
I recall at least reports for aioredis, aiomysql and aiokafka for these
deprecation warnings.
Sure, fix is trivial (and will be applied to next release of libs) but I pretty
sure there are many other usages.
--
ver
Andrew Svetlov added the comment:
Done.
Let's keep the issue open until the documentation will be committed
--
___
Python tracker
<https://bugs.python.org/is
Andrew Svetlov added the comment:
Thanks for letting me know.
Investigating.
BrokenPipeError is not related, I had a plan to fix such messages during the
beta phase.
--
___
Python tracker
<https://bugs.python.org/issue36
Andrew Svetlov added the comment:
A champion is welcome :)
--
___
Python tracker
<https://bugs.python.org/issue36373>
___
___
Python-bugs-list mailing list
Unsub
Andrew Svetlov added the comment:
I would say that if requests a designed from scratch more idiomatic way could be
with requests.Session() as session:
session.get('https://example.com/')
or
session = requests.Session()
session.get('https://example.com/')
sessi
Andrew Svetlov added the comment:
decimal was changed from threading.local to contextvar usage.
The module is "safe" not only for asyncio but for threading, trio etc.
unittest.mock doesn't use explicit context all for patching.
It changes global interpreter-wide objec
Change by Andrew Svetlov :
--
resolution: -> rejected
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue33918>
___
___
Andrew Svetlov added the comment:
Thank you.
The change is pretty straightforward.
There is no need to jumbo PR, you can split the work into PR-per-module if it
is more comfortable to you.
I'm ok with reviewing any approach.
--
___
P
Change by Andrew Svetlov :
--
versions: +Python 3.8 -Python 3.7
___
Python tracker
<https://bugs.python.org/issue36373>
___
___
Python-bugs-list mailin
Change by Andrew Svetlov :
--
title: asyncio.gather: no docs for deprecated loop parameter -> Deprecate
explicit loop parameter in all public asyncio APIs
___
Python tracker
<https://bugs.python.org/issu
Change by Andrew Svetlov :
--
title: asyncio.gather: no docs for deprecated loop parameter -> Deprecate
explicit loop parameter in all public asyncio APIs
___
Python tracker
<https://bugs.python.org/issu
Change by Andrew Svetlov :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Andrew Svetlov added the comment:
Wow!
Thanks for the report.
Internally it is pretty close to freebsd problem.
The test is unstable, not asyncio code itself.
The test uses future objects to synchronize client and server socket processing
but looks like there are race conditions still
Andrew Svetlov added the comment:
Fixed
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
1001 - 1100 of 2519 matches
Mail list logo