[issue32677] Add str.isascii()

STINNER Victor added the comment: > (Voting on python-ideas ML now) https://mail.python.org/pipermail/python-ideas/2018-January/048737.html -- nosy: +vstinner ___ Python tracker

[issue32677] Add str.isascii()

New submission from INADA Naoki : int() and str.is*** functions accepts other than ASCII. But we want to accept only ASCII in some cases. (e.g. ipaddress module) We can use try-except to check ASCII, but it's inefficient. try: s.encode('ascii') except

[issue32677] Add str.isascii()

Change by INADA Naoki : -- keywords: +patch pull_requests: +5188 stage: -> patch review ___ Python tracker ___

[issue32662] Implement Server.serve_forever and corresponding APIs

STINNER Victor added the comment: The test fails differently on s390x SLES 3.x: http://buildbot.python.org/all/#/builders/16/builds/576 ... test_force_close (test.test_asyncio.test_selector_events.SelectorTransportTests) ... ok test_set_nodelay

[issue32662] Implement Server.serve_forever and corresponding APIs

STINNER Victor added the comment: test_asyncio.test_start_server_1() hangs on "s390x Debian 3.x" buildbot: http://buildbot.python.org/all/#builders/13/builds/579 test_set_nodelay (test.test_asyncio.test_selector_events.TestSelectorUtils) ... ok test_start_server_1

[issue30622] Fix NPN guard for OpenSSL 1.1

Change by Roundup Robot : -- pull_requests: +5189 ___ Python tracker ___

[issue32675] dict.__contains__(unhashable) raises TypeError where False was expected

INADA Naoki added the comment: "Key must be hashable" is precondition of dict operations. All operations for dict will raise TypeError for unhashable keys. For example, d[k] will raise TypeError too, but document doesn't mention about it: """ Return the item of d with

[issue30622] Fix NPN guard for OpenSSL 1.1

Change by Roundup Robot : -- pull_requests: +5187 ___ Python tracker ___

[issue32675] dict.__contains__(unhashable) raises TypeError where False was expected

New submission from xitop : The `dict.__contains()` function does not fully comply with the description in the "Data Model" section of the official documentation, which states: "__contains__(self, item) ... Should return true if item is in self, false otherwise." Basically

[issue32676] test_asyncio emits many warnings when run in debug mode

New submission from STINNER Victor : vstinner@apu$ ./python -u -X dev -m test -v test_asyncio 2>&1|tee log Tests which emit warnings: test__sock_sendfile_native_failure (test.test_asyncio.test_base_events.BaseLoopSendfileTests) ...

[issue32662] Implement Server.serve_forever and corresponding APIs

STINNER Victor added the comment: Just to be complete, the test also fails on x86 Gentoo Non-Debug with X 3.x: http://buildbot.python.org/all/#/builders/99/builds/564 ... test_start_server_1 (test.test_asyncio.test_server.SelectorStartServerTests) ... Exception in

[issue32441] os.dup2 should return the new fd

STINNER Victor added the comment: The change introduced a warning. Can someone please take a look (and maybe fix it)? ./Modules/posixmodule.c: In function 'os_dup2_impl': ./Modules/posixmodule.c:7785:9: warning: 'res' may be used uninitialized in this function

[issue32378] test_npn_protocols broken with LibreSSL 2.6.1+

Bernard Spil added the comment: See also https://github.com/python/cpython/pull/5343 Note: I am Sp1l on Github, spil here, CLA has been signed. -- nosy: +spil pull_requests: +5190 ___ Python tracker

[issue32513] dataclasses: make it easier to use user-supplied special methods

Eric V. Smith added the comment: Here's where I am on what methods are added under which circumstances. Hopefully these tables don't get mangled. The more I think about what to do with hash and raising exceptions, the more I think that the right answer is to not raise any

[issue32678] Lazy import ast in inspect

Change by INADA Naoki : -- keywords: +patch pull_requests: +5191 stage: -> patch review ___ Python tracker ___

[issue2793] Dictionary fails to index when adding list when in a deeply nested loop

Change by Cheryl Sabella : -- pull_requests: -5177 ___ Python tracker ___ ___

[issue27931] Email parse IndexError <""@wiarcom.com>

Jay Yin added the comment: https://github.com/python/cpython/pull/5329 this is the pending pull request for this -- nosy: +jayyin11043 ___ Python tracker

[issue32679] concurrent.futures should store full sys.exc_info()

Jonas H. added the comment: See also https://stackoverflow.com/questions/19309514/getting-original-line-number-for-exception-in-concurrent-futures for other people having the same problem -- ___ Python tracker

[issue32680] smtplib SMTP instances missing a default sock attribute

Change by Romuald Brunet : -- keywords: +patch pull_requests: +5192 stage: -> patch review ___ Python tracker ___

[issue32680] smtplib SMTP instances missing a default sock attribute

Romuald Brunet added the comment: My use case: try: s = SMTP('myhost') s.do_some_sending() finaly: if s is not None and s.sock is not None: s.quit() But I realize just now that in that case, if s was initialized correctly, its sock was inevitably set

[issue32681] Fix uninitialized variable in os_dup2_impl

New submission from Stéphane Wirtel : With the last revision, I get this warning from the compiler. gcc -pthread -Wno-unused-result -Wsign-compare -g -Og -Wall -Wstrict-prototypes -std=c99 -Wextra -Wno-unused-result -Wno-unused-parameter -Wno-missing-field-initializers

[issue18533] Avoid error from repr() of recursive dictview

Senthil Kumaran added the comment: I merged this one. All the requirements were met. @Terry / @ Cheryl - could you please review if I did it right? Should this be backported to other Python versions? -- resolution: -> fixed versions: +Python 3.7 -Python 2.7,

[issue32680] smtplib SMTP instances missing a default sock attribute

New submission from Romuald Brunet : SMTP instances from the smtplib module are not creating their sock attribute consistently after __init__ When host is sent as parameter a sock object is created (and hopefully, connected) When the host is not sent, the sock

[issue32681] Fix uninitialized variable in os_dup2_impl

STINNER Victor added the comment: I already reported the issue at: https://bugs.python.org/issue32441#msg310752 I suggest to close this issue as a duplicate. -- nosy: +vstinner ___ Python tracker

[issue32681] Fix uninitialized variable in os_dup2_impl

Change by Stéphane Wirtel : -- keywords: +patch pull_requests: +5194 stage: -> patch review ___ Python tracker ___

[issue23080] BoundArguments.arguments should be unordered

Cheryl Sabella added the comment: I know there are other issues to remove OrderedDicts, so should this patch be used for BoundArguments? -- nosy: +csabella ___ Python tracker

[issue23080] BoundArguments.arguments should be unordered

Change by Cheryl Sabella : -- stage: -> needs patch type: -> enhancement versions: +Python 3.7 -Python 3.5 ___ Python tracker ___

[issue18533] Avoid error from repr() of recursive dictview

Senthil Kumaran added the comment: New changeset d7773d92bd11640a8c950d6c36a9cef1cee36f96 by Senthil Kumaran (bennorth) in branch 'master': bpo-18533: Avoid RecursionError from repr() of recursive dictview (#4823)

[issue32679] concurrent.futures should store full sys.exc_info()

New submission from Jonas H. : Use case: Try to get a future's result using concurrent.futures.Future.result(), and log the full exception if there was any. Currently, only "excinst" (sys.exc_info()[1]) is provided with the Future.exception() method. Proposal: Add new

[issue32681] Fix uninitialized variable in os_dup2_impl

Stéphane Wirtel added the comment: Yes, you can close it, but I don't understand why there is no fix for this warning. my issue has a small PR, but the other issue on b.p.o, I am not really sure about my fix... I just set the 'res' variable to 0 and avoid this kind of

[issue32665] pathlib.Path._from_parsed_parts should call cls.__new__(cls)

qb-cea added the comment: Typo in the code of the previous comment: - __slots__ = ("new_attr",) + __slots__ = ("_new_attr",) -- type: -> behavior ___ Python tracker

[issue32678] Lazy import ast in inspect

New submission from INADA Naoki : inspect imports ast but it's used only for creating signature. But asyncio uses inspect module for unwrap(), isgenerator(), etc... And ast module is relatively heavy for import. This is output of python -Ximporttime -c 'import asyncio':

[issue32640] Python 2.7 str.join documentation is incorrect

Mariatta Wijaya added the comment: New changeset f5e8f71fe339fe5e47c4d24ac2e5177fa3c02922 by Mariatta in branch '2.7': [2.7] bpo-32640: Clarify the behavior of str.join and unicode object (GH-5333)

[issue32640] Python 2.7 str.join documentation is incorrect

Mariatta Wijaya added the comment: Thanks! -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue32677] Add.isascii() to str, bytes and bytearray

Change by INADA Naoki : -- title: Add str.isascii() -> Add.isascii() to str, bytes and bytearray ___ Python tracker ___

[issue32682] test_zlib improve version parsing

New submission from pmpp : On some OS like android 4.0+, system libz.so gives funny versions number that prevent test_zlib from correctly parse it from zlib.ZLIB_RUNTIME_VERSION https://patch-diff.githubusercontent.com/raw/vstinner/cpython/pull/1.patch seems to fix

[issue32680] smtplib SMTP instances missing a default sock attribute

Stéphane Wirtel added the comment: You are right, but... if there is an exception in the connect method before the init of self.sock, the sock attribute will not exist and in this case, and only in this case, there will be an exception in the close method. your code will

[issue32206] Run modules with pdb

Change by Jason R. Coombs : -- nosy: +jason.coombs ___ Python tracker ___ ___

[issue3177] Add shutil.open

Change by Matej Cepl : -- nosy: +mcepl ___ Python tracker ___ ___ Python-bugs-list mailing

[issue25330] Docs for pkgutil.get_data inconsistent with semantics

Paul Ganssle added the comment: I'm not sure if this warrants a separate issue, but I also notice this in the documentation: > If the package cannot be located or loaded, or it uses a loader which does > not support get_data, then None is returned. In particular, the

[issue32682] test_zlib improve version parsing

Change by pmpp : -- keywords: +patch pull_requests: +5195 stage: -> patch review ___ Python tracker ___

[issue32681] Fix uninitialized variable in os_dup2_impl

Change by Stéphane Wirtel : -- versions: +Python 3.7 ___ Python tracker ___ ___

[issue32513] dataclasses: make it easier to use user-supplied special methods

Eric V. Smith added the comment: I'm working on a better formulation of the __hash__ issue, so don't invest much time in this until I post my next message. I'm going to re-organize the various tables to make them easier to read. --

[issue32513] dataclasses: make it easier to use user-supplied special methods

Guido van Rossum added the comment: Can you also clarify that this *never* looks at whether the method is implemented in a superclass? And clarify what happens with the __hash__ = None that's automatically created when you define __eq__. Please also repeat the default value

[issue32579] UUID module fix, uuid1 python module function

Change by Ned Deily : -- nosy: +barry stage: -> patch review ___ Python tracker ___ ___

[issue32493] UUID Module - FreeBSD build failure

Change by Barry A. Warsaw : -- nosy: +barry ___ Python tracker ___ ___ Python-bugs-list

[issue32206] Run modules with pdb

Jason R. Coombs added the comment: Thanks for creating this functionality. I'm so excited to use it that I've published a backport [https://pypi.org/project/backports.pdb], which seems to work for at least the basic use cases on Python 2.7+. --

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

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)

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

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's

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

Change by Yury Selivanov : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue18533] Avoid error from repr() of recursive dictview

Change by Roundup Robot : -- pull_requests: +5196 ___ Python tracker ___

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

Guido van Rossum added the comment: OK. I hope both of you keep an eye on this and actively try to test and break it! -- ___ Python tracker

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

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

[issue18533] Avoid error from repr() of recursive dictview

Terry J. Reedy added the comment: Thank you for checking and merging. In the future, we will be reminded to change '#' to 'GH-' in the title. I only found out a couple of days ago that this is desired. David, in msg193651, and Raymond, in msg193779, already suggested

[issue32662] Implement Server.serve_forever and corresponding APIs

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

[issue32630] Migrate decimal to use PEP 567 context variables

Guido van Rossum added the comment: Guys. Please stop with the editorializing. "I cannot believe ..." (used essentially by both of you) is not constructive. -- ___ Python tracker

[issue32446] ResourceLoader.get_data() should accept a PathLike

Jay Yin added the comment: this would extend to set and get data right?, also would it be good to use the current get_data() (one that reads only string) function for the overloaded one and extract the string path from the os.path object? or would we want the os.path

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: Guido, I apologize for the outburst. I had the impression that msg310799 implicitly asserted my incompetence in benchmarking. -- ___ Python tracker

[issue32630] Migrate decimal to use PEP 567 context variables

Elvis Pranskevichus added the comment: Likewise, on the same builds, running _decimal/tests/bench.py does not show a significant difference: https://gist.github.com/elprans/fb31510ee28a3aa091aee3f42fe65e00 -- ___ Python tracker

[issue15115] Duplicated Content-Transfer-Encoding header when applying email.encoders

Change by Cheryl Sabella : -- keywords: +patch pull_requests: +5200 stage: needs patch -> patch review ___ Python tracker ___

[issue32571] Speed up and clean up getting optional attributes in C code

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

[issue32545] Unable to install Python 3.7.0a4 on Windows 10 - Error 0x80070643: Failed to install MSI package.

Terry J. Reedy added the comment: Can one of the Windows experts look at the attached log? -- ___ Python tracker ___

[issue32615] Inconsistent behavior if globals is a dict subclass

ppperry added the comment: Uh, I'm not undertsanding the relevance of whether the code is run at module-level or not. It can't possibly be a feature that * some code * uses the overrides and *other code* doesn't. -- ___ Python

[issue32615] Inconsistent behavior if globals is a dict subclass

Terry J. Reedy added the comment: I was referring back to your original a) and b) items. Perhaps 'accidental' versus 'intentional, for whatever reason', would be the better contrast. The weekly new issue report comes out on Fridays around noon eastern US. I hope someone

[issue32616] Significant performance problems with Python 2.7 built with clang 3.x or 4.x

Change by Terry J. Reedy : -- components: +macOS nosy: +ned.deily, ronaldoussoren ___ Python tracker ___

[issue32436] Implement PEP 567

Change by Yury Selivanov : -- pull_requests: +5197 ___ Python tracker ___ ___

[issue32446] ResourceLoader.get_data() should accept a PathLike

Jay Yin added the comment: I'm taking a look now, this looks interesting -- nosy: +jayyin11043 ___ Python tracker ___

[issue32614] Fix documentation examples of using re with escape sequences

Terry J. Reedy added the comment: Definitely. x.y code, including doc examples, should free of such warnings. -- nosy: +terry.reedy ___ Python tracker

[issue32626] Subscript unpacking raises SyntaxError

Terry J. Reedy added the comment: Tuples often need parentheses when embedded in expressions. So often that beginners often think that they are required. I believe this should be closed as 'not a bug'. I nosied the two authors of the implementation for their opinion.

[issue32545] Unable to install Python 3.7.0a4 on Windows 10 - Error 0x80070643: Failed to install MSI package.

Matt Ripley added the comment: Hi Terry, Thanks for getting back to me on this. I tried all of the suggestions on the link that you sent to me and the python installer still will not install for me. I get the same errors. I can install the conda python distributions.

[issue32630] Migrate decimal to use PEP 567 context variables

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: >

[issue32613] Use PEP 397 py launcher in windows faq

Change by Terry J. Reedy : -- components: +Windows nosy: +paul.moore, steve.dower, tim.golden, zach.ware ___ Python tracker ___

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: I have run about 1000 times more decimal benchmarks than both Yury and you. You attempt to hurt my reputation is laughable. Show me some top-performance code that you have written. -- ___ Python

[issue32630] Migrate decimal to use PEP 567 context variables

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.

[issue32630] Migrate decimal to use PEP 567 context variables

Guido van Rossum added the comment: Stefan this is unacceptable abuse. Please read the code of conduct. -- ___ Python tracker ___

[issue32615] Inconsistent behavior if globals is a dict subclass

Terry J. Reedy added the comment: I did take the code, and it would not run because of the indent. With that fixed, it still did not run because the the indent in the quoted code fed to exec. This does run: import builtins class K(dict): def __getitem__(self, k):

[issue32521] NIS module fails to build due to the removal of interfaces related to Sun RPC from glibc.

Change by Christian Heimes : -- pull_requests: +5198 ___ Python tracker ___ ___

[issue32630] Migrate decimal to use PEP 567 context variables

STINNER Victor added the comment: > Likewise, on the same builds, running _decimal/tests/bench.py does not show a > significant difference: > https://gist.github.com/elprans/fb31510ee28a3aa091aee3f42fe65e00 Note: it may be interesting to rewrite this benchmark my

[issue32630] Migrate decimal to use PEP 567 context variables

Guido van Rossum added the comment: Stefan, I don't think a module author should retain veto over everything affecting their code forever. (We've had spectacular process failures with this in the past.) Please take a deep breath and patiently answer Yury's questions. If you

[issue32446] ResourceLoader.get_data() should accept a PathLike

Barry A. Warsaw added the comment: Hi Jay. Thanks for your interest in contributing to Python! Actually, I'm going to close this issue since we ultimately decided to deprecate the ResourceLoader ABC:

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: On Fri, Jan 26, 2018 at 09:06:38PM +, Yury Selivanov wrote: > This benchmark is specially constructed to profile creating decimal contexts > and doing almost nothing with the It is not constructed at all. It was the first thing I wrote

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: Yury, would you be willing to work this out by email? -- I think it was you who I discussed the context-subclassing with and that was quite a pleasant experience. -- ___ Python tracker

[issue32630] Migrate decimal to use PEP 567 context variables

STINNER Victor added the comment: Since the root of the discussion is a performance regression, let me take a look since I also care of not regressing in term of performance. We (CPython core developers, as as team) spent a lot of time on optimizing CPython to make

[issue32615] Inconsistent behavior if globals is a dict subclass

Change by pmpp : -- nosy: +pmpp ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: Tests - I ran some of my own tests (not even close to all), they seem fine. 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. Performance ---

[issue32446] ResourceLoader.get_data() should accept a PathLike

Jay Yin added the comment: I hope this isn't a dumb question but where is ResourceLoader.get_data() implemented, using ATOM to search the whole project and I can't seem to find it. -- ___ Python tracker

[issue32436] Implement PEP 567

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/226e50049da43097d89e9e99f9c55f212d1f7bd5

[issue32630] Migrate decimal to use PEP 567 context variables

Elvis Pranskevichus added the comment: FWIW, I ran bm_telco with pyperformance on a benchmark-tuned system and did not observe the slowdown. Benchmarks were done on a release build (--enable-optimizations) $ sudo (which python3) -m perf system tune MASTER: $ pyperformance

[issue15115] Duplicated Content-Transfer-Encoding header when applying email.encoders

Cheryl Sabella added the comment: Hi David, I've made a pull request for the way I think you wanted this documented. Please take a look and let me know if it's even close to what you were suggesting. Thanks! :-) -- stage: patch review -> needs patch

[issue32621] Problem of consistence in collection.abc documentation

Change by Terry J. Reedy : -- nosy: +rhettinger ___ Python tracker ___ ___

[issue32621] Problem of consistence in collection.abc documentation

Change by Terry J. Reedy : -- assignee: -> docs@python components: +Documentation, Library (Lib) nosy: +docs@python stage: -> needs patch ___ Python tracker

[issue32626] Subscript unpacking raises SyntaxError

Ben Burrill added the comment: Yeah, but in this case, you don't need parentheses unless you use unpacking. That is unexpected behavior. -- ___ Python tracker

[issue32670] Enforce PEP 479—StopIteration and generators—in Python 3.7 by default

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/43c47fe09640c579462978ec16f81295f5857cde --

[issue32670] Enforce PEP 479—StopIteration and generators—in Python 3.7 by default

Change by Yury Selivanov : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue32683] isinstance is calling ob.__getattribute__ as a fallback instead of object.__class__.__get__

New submission from Dan Snider : It doesn't even care if the result it gets from ob.__getattribute__("__class__") isn't a type. Surely `isinstance` is intended to do what it is named and be guaranteed to return True if an object `ob` is an instance of class `cls`.

[issue32611] Tkinter taskbar icon (Windows)

Terry J. Reedy added the comment: On Win10, with 3.5.4, 3.6.4, 3.7.0a4, the pinned IDLE icons on my taskbar look normal, with the notebook and snake icon. Running import tkinter as tk r = tk.Tk() stacks the edge of a shadow icon behind the IDLE icon. When I mouse over the

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: Guido, I have the feeling that the feature -- about which I was actually positive in the first place -- is being pushed aggressively with no respect for the module author. BTW, prec is changed quite frequently in decimal code, so if people

[issue32446] ResourceLoader.get_data() should accept a PathLike

Jay Yin added the comment: nvm I temporarily forgot I was working with python here, I'd need to use "is" to check what the entered "path" is huh? -- ___ Python tracker

[issue32675] dict.__contains__(unhashable) raises TypeError where False was expected

Change by R. David Murray : -- resolution: -> not a bug stage: -> resolved status: open -> closed ___ Python tracker ___

[issue32630] Migrate decimal to use PEP 567 context variables

Guido van Rossum added the comment: Apologies accepted. I did not imply that -- I was simply stating that Yury needed your help reproducing your result so he could do something about it. It seems you two are taking this offline so I trust that there will be no more barbs.

  1   2   >