[issue45520] Frozen dataclass deep copy doesn't work with __slots__

2021-10-20 Thread Justin Furuness
Justin Furuness added the comment: Thank you for the in-depth explanation. That all makes sense to me, I have run into the __slots__ with defaults issues before, I'll be sure to try out these fixes. I appreciate you taking the time. Thanks, Justin On Tue, Oct 19, 2021 at 5:28 PM Josh

[issue45520] Frozen dataclass deep copy doesn't work with __slots__

2021-10-19 Thread Justin Furuness
Justin Furuness added the comment: I didn't realize it was possible to add slots in that way (slots=True). I think for normal classes manually defining __slots__ is the common way of adding __slots__ (at least for me), so it feels weird to me that manually defining __slots__ instead of adding

[issue43758] dict.config TimedRotatingFileHandler filename .suffix does not function

2021-04-06 Thread Justin Lee
New submission from Justin Lee : Hi, I would like to change the .suffix for the TimedRotatingFileHandler for the "future-created" file in dictConfig written in json. Is there any suggestion on how to do that? -- messages: 390400 nosy: tea940314 priority: normal severi

[issue42695] tkinter keysym_num value is incorrect

2020-12-20 Thread Justin
Justin added the comment: TK bug ticket has been created at https://core.tcl-lang.org/tk/tktview/ffe6925b916caac02acae53f745e95dd1c557019 -- ___ Python tracker <https://bugs.python.org/issue42

[issue42695] tkinter keysym_num value is incorrect

2020-12-20 Thread Justin
New submission from Justin : Hi there. On my MacOS 10.14.16 laptop with a qwerty keyboard I was testing tkinter keyboard listening for an azerty keyboard layout by switching the layout to `French - PC` When I press qwerty keys Shift + \ I expect to see 'μ' printed. When looking at the tkinter

[issue42640] tkinter throws exception when key is pressed

2020-12-20 Thread Justin
Justin added the comment: Thank you very much. I understand and just wanted to let you know. In brew I opened up this ticket(https://github.com/Homebrew/homebrew-core/issues/67327) with that team. -- ___ Python tracker <https://bugs.python.

[issue42640] tkinter throws exception when key is pressed

2020-12-20 Thread Justin
Justin added the comment: FYI, I just brew installed python and with: ``` Python 3.9.1 (default, Dec 17 2020, 03:56:09) [Clang 11.0.0 (clang-1100.0.33.17)] on darwin ``` This issue still happens -- ___ Python tracker <https://bugs.python.

[issue42640] tkinter throws exception when key is pressed

2020-12-14 Thread Justin
Justin added the comment: Thanks for responding so quickly. 1. My python version is: Python 3.8.6 (default, Oct 8 2020, 14:07:53) [Clang 11.0.0 (clang-1100.0.33.17)] on darwin 2. N/A 3. >>> from tkinter.test.support import * >>> get_tk_patchlevel() (8, 5, 9, 'final', 0)

[issue42640] tkinter throws exception when key is pressed

2020-12-14 Thread Justin
Justin added the comment: Sorry my verification keyboard listener program is not the pygame one. It is this one: ``` from tkinter import * def keyup(e): print('up', e.__dict__) def keydown(e): print('down', e.__dict__) root = Tk() frame = Frame(root, width=100, height=100) frame.bind

[issue42640] tkinter throws exception when key is pressed

2020-12-14 Thread Justin
New submission from Justin : On macOs 10.14.6 laptop with a qwerty layout, when I switch my keyboard layout to `French - PC` and use a tkinter keyboard listener and press the '[' button which should be the '^' button on the azerty keyboard, this Error is thrown: ``` 2020-12-14 10:13:26.533

[issue13337] IGNORE_CASE doctest option flag

2020-11-28 Thread Justin Baum
Change by Justin Baum : -- keywords: +patch nosy: +justinba1010 nosy_count: 3.0 -> 4.0 pull_requests: +22426 stage: -> patch review pull_request: https://github.com/python/cpython/pull/23547 ___ Python tracker <https://bugs.python.org/i

[issue42140] asyncio.wait function creates futures set two times

2020-10-26 Thread Justin Arthur
Justin Arthur added the comment: > So the behaviour you describe will happen anyway. Unless we make your change against the 3.9 branch, in which case the fix will make it into the 3.9.x series of patch releases. -- ___ Python tracker <

[issue42140] asyncio.wait function creates futures set two times

2020-10-26 Thread Justin Arthur
Justin Arthur added the comment: I believe the documentation may be referring to the English set and not a Python set, but I could be wrong. Yury changed the wording from sequence to set in 3.7, but we didn't document a breaking change as far as I know. The purpose of those set

[issue42140] asyncio.wait function creates futures set two times

2020-10-26 Thread Justin Arthur
Justin Arthur added the comment: Your change makes perfect sense to me. It would be a backport-only change as the 2nd set creation is actually getting removed for the development version (3.10) to finalize the deprecation of wait's coroutine scheduling. -- nosy: +JustinTArthur

[issue38912] test_asyncio altered the execution environment

2020-10-13 Thread Justin Arthur
Justin Arthur added the comment: The "'NoneType' object has no attribute 'close'" error is likely caused by a race against the loop calling the test protocol object's connection_made callback. I was able to reproduce this case occasionally on macOS and it's likely platform-agnos

[issue38912] test_asyncio altered the execution environment

2020-10-13 Thread Justin Arthur
Change by Justin Arthur : -- nosy: +JustinTArthur nosy_count: 5.0 -> 6.0 pull_requests: +21662 pull_request: https://github.com/python/cpython/pull/22691 ___ Python tracker <https://bugs.python.org/issu

[issue33533] Provide an async iterator version of as_completed

2020-10-08 Thread Justin Arthur
Change by Justin Arthur : -- versions: +Python 3.10 -Python 3.8 ___ Python tracker <https://bugs.python.org/issue33533> ___ ___ Python-bugs-list mailin

[issue33533] Provide an async iterator version of as_completed

2020-10-02 Thread Justin Arthur
Change by Justin Arthur : -- title: Provide an async-generator version of as_completed -> Provide an async iterator version of as_completed ___ Python tracker <https://bugs.python.org/issu

[issue33533] Provide an async-generator version of as_completed

2020-10-02 Thread Justin Arthur
Justin Arthur added the comment: Thanks, Hrvoje. I've updated the patch to match this bug's suggested format and have updated the documentation and What's New. The one quirk that comes with the benefit of getting the same futures back is that we still allow both coroutines and futures

[issue38032] lib2to3 doesn't parse Python 3 identifiers containing non-spacing marks

2020-10-01 Thread Justin Arthur
Justin Arthur added the comment: Not sure if there is consensus on how to fix, but fixing #12731 will fix this for most of the cases I've seen complaints about as a side effect. -- ___ Python tracker <https://bugs.python.org/issue38

[issue33533] Provide an async-generator version of as_completed

2020-10-01 Thread Justin Arthur
Justin Arthur added the comment: I've added a new PR, PR 22491. This one has as_completed returning an iterator and includes tests for both the old and new style. I see a trivial amount of latency added in extra call stack over Guido's original implementation. Should we decide to deprecate

[issue33533] Provide an async-generator version of as_completed

2020-10-01 Thread Justin Arthur
Change by Justin Arthur : -- nosy: +JustinTArthur nosy_count: 5.0 -> 6.0 pull_requests: +21508 pull_request: https://github.com/python/cpython/pull/22491 ___ Python tracker <https://bugs.python.org/issu

[issue41337] strangnedd with the parser

2020-07-19 Thread Justin Hodder
New submission from Justin Hodder : Here a colab that demostrates the bug https://colab.research.google.com/drive/1OWSEoV7Wx-EBA_2IprNZoASNvXIky3iC?usp=sharing the following code gives"received an invalid combination of arguments - got (Tensor, Tensor), but expected one of:" im

[issue40078] asyncio subprocesses allow pids to be reaped, different behavior than regular subprocesses

2020-03-26 Thread Justin Lebar
Change by Justin Lebar : -- type: -> behavior versions: +Python 3.9 ___ Python tracker <https://bugs.python.org/issue40078> ___ ___ Python-bugs-list mai

[issue40078] asyncio subprocesses allow pids to be reaped, different behavior than regular subprocesses

2020-03-26 Thread Justin Lebar
Change by Justin Lebar : -- versions: +Python 3.7, Python 3.8 ___ Python tracker <https://bugs.python.org/issue40078> ___ ___ Python-bugs-list mailing list Unsub

[issue1187312] subprocess: optional auto-reaping fixing os.wait() lossage

2020-03-26 Thread Justin Lebar
Justin Lebar added the comment: 15 years later, it seems asyncio subprocesses may have the same issue. :) https://bugs.python.org/issue40078 -- nosy: +Justin.Lebar ___ Python tracker <https://bugs.python.org/issue1187

[issue40078] asyncio subprocesses allow pids to be reaped, different behavior than regular subprocesses

2020-03-26 Thread Justin Lebar
New submission from Justin Lebar : >From https://bugs.python.org/issue1187312 about regular subprocesses: > So as long as the application keeps a reference to the > subprocess object, it can wait for it; auto-reaping only > starts when the last reference was dropped [in P

[issue38860] GenericPyCData_new does not invoke new or init

2020-01-22 Thread Justin Capella
Justin Capella added the comment: Becca were you still interested in this issue? Anyone agree the unit test seems to have a bug if that is intended behavior? Is there a better forum for discussion about design/behavior, maybe the more generic issue of GenericPyCData_new using tp_alloc

[issue39175] Funkness with issubset

2020-01-01 Thread Justin Hodder
Justin Hodder added the comment: AttributeError: 'tuple' object has no attribute 'issubset' And it doesn't explain why it works as expected in Brython. -- ___ Python tracker <https://bugs.python.org/issue39

[issue39175] Funkness with issubset

2019-12-31 Thread Justin Hodder
Justin Hodder added the comment: oh I'm using Python 3.8.1 (32-bit) 3.8.1150.0 on win10 -- ___ Python tracker <https://bugs.python.org/issue39175> ___ ___ Pytho

[issue39175] Funkness with issubset

2019-12-31 Thread Justin Hodder
Justin Hodder added the comment: This works: $ diff PythonDoesntWorks.py HowCanYouPayMana.v3.py 60c60 < if x2.issubset(set(avalMana.keys())): --- > if x2.issubset(set(list(avalMana.keys(: 62a63 > this doesn't work: $ diff PythonDoesntWorks.py HowCanYouPayMana.v3

[issue39175] Funkness with issubset

2019-12-31 Thread Justin Hodder
New submission from Justin Hodder : line 59,"print(x2,"avalMana",set(avalMana.keys()))" prints:"{('A', 'B')} avalMana {'A', ('A', 'B'), ('A', 'C')}" line 60,"if x2.issubset(set(avalMana.keys())):" is False change line 60 to "

[issue38860] GenericPyCData_new does not invoke new or init

2019-12-11 Thread Justin Capella
Justin Capella added the comment: It might make sense that CSimpleData types do not call init, which makes sense for _fields_ but it doesn't make sense to offer subclassing (of Structure) and not use the subclass, instead creating a type of the same name. That test case is bugged, missing

[issue38576] CVE-2019-18348: CRLF injection via the host part of the url passed to urlopen()

2019-11-20 Thread Justin Capella
Justin Capella added the comment: Can't see the specifics of that "restricted" redhat bug, but this was interesting bug and I wanted to ask if perhaps the domain in such cases should be IDN / punycoded ://xn--n28h.ws/ for example is ://.la -- nosy:

[issue38860] GenericPyCData_new does not invoke new or init

2019-11-19 Thread Justin Capella
New submission from Justin Capella : When subclassing the ctypes.Structure class, __new__ and __init__ are not invoked when using the inherited classmethod from_buffer_copy to create the object. I think this is because tp_alloc is ultimately used by GenericPyCData_new when creating

[issue38166] ast identifies incorrect column for compound attribute lookups

2019-09-13 Thread Justin McCann
New submission from Justin McCann : This issue is related to https://github.com/microsoft/vscode-python/issues/7327 and https://github.com/PyCQA/pylint/issues/3103 For compound attribute access in variable references like a.b.c.d, the AST reports the column of the first variable/attribute

[issue12731] python lib re uses obsolete sense of \w in full violation of UTS#18 RL1.2a

2019-09-07 Thread Justin Arthur
Change by Justin Arthur : -- nosy: +JustinTArthur ___ Python tracker <https://bugs.python.org/issue12731> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue38032] lib2to3 doesn't parse Python 3 identifiers containing non-spacing marks

2019-09-05 Thread Justin Arthur
Justin Arthur added the comment: Ned, can you confirm that 2to3 is not intended for cumulative/incremental runs over the same codebase? If it's not intended to be run on previously ported code, this will just need to be fixed on the lib2to3 downstream projects like awpa and Black

[issue38032] lib2to3 doesn't parse Python 3 identifiers containing non-spacing marks

2019-09-04 Thread Justin ARthur
New submission from Justin ARthur : Python 3 code with an identifier that has a non-spacing mark in it does not get tokenized by lib2to3 and will result in an exception thrown in the parsing process. Parsing the attached file (badvar.py), results in `ParseError: bad token: type=58, value

[issue37372] datetime.time unpickling fails in case of python2 pickle with seconds>=24

2019-06-22 Thread Justin Blanchard
New submission from Justin Blanchard : Under bpo-22005, Python 3 gained support for unpickling datetime data from Python 2. (Thanks!) It turns out the input validation isn't quite right: it bombs on datetime.time when the encoded seconds (not hours) field is >=24: python2>>> dat

[issue37086] time.sleep error message misleading

2019-05-30 Thread Justin Fay
Justin Fay added the comment: >From looking at the code for this (note I am not a C programmer so may have >gotten this wrong) _PyTime_FromObject first checks if the object is a float >using PyFloat_Check(obj) this is evident as passing nan to time.sleep raises a &g

[issue37086] time.sleep error message misleading

2019-05-29 Thread Justin Fay
New submission from Justin Fay : Using python3.6 and calling `time.sleep` with an invalid argument the `TypeError` raised has the error message "TypeError: an integer is required". This is not the case as a float or integer is acceptable. Using python 2.7 the error message given

[issue36931] json lib doesnt want to load from file

2019-05-15 Thread Justin Rose
New submission from Justin Rose : when I run the included file i get an error that looks like: Traceback (most recent call last): File "/home/justin/Desktop/pkmn/main.py", line 10, in expansion = json.load(expan_list) File "/usr/lib/python3.6/json/__init__.py"

[issue35531] xml.etree.ElementTree Elment.find bug, fails to find tag

2018-12-18 Thread Justin
Justin added the comment: Issue was user error. I though that find did a full search of the tree when it only searches children. Solution is: ele = xml.find('.//faultstring') -- resolution: -> not a bug stage: -> resolved status: open -&g

[issue35531] xml.etree.ElementTree Elment.find bug, fails to find tag

2018-12-18 Thread Justin
New submission from Justin : When the following text it loaded in to an ElementTree Element, the find method is unable to find one of the elements without a namespace assigned to it. ``` import xml.etree.ElementTree as ElementTree xml_text = """ http://schemas.xmlsoap.or

[issue33643] Mock functions with autospec STILL don't support assert_called_once, assert_called, assert_not_called

2018-10-08 Thread Justin Dray
Justin Dray added the comment: I can still reproduce this with python 3.6.5: https://gist.github.com/justin8/c6a565d3b64222a9ba3a2568d1d830ee no .assert_called() on autospec'd functions, but it works with normal mock.MagicMock() ipython autocomplete shows assert_any_call

[issue27755] Retire DynOptionMenu with a ttk Combobox

2017-07-14 Thread Justin Foo
Justin Foo added the comment: Thanks for analysing further, Terry. My original goal was a drop-in replacement for an aspect of IDLE I found quite annoying (on Windows). Unfortunately, my limited playing with tkinter didn't inspire me to go beyond

[issue29837] python3 pycopg2 import issue on solaris 10

2017-03-17 Thread justin
New submission from justin: Hi, I have installed psycopg2 through pip3, but when I tried to import it, I got the following error. what could be the problem? help> psycopg2 problem in psycopg2 - ImportError: ld.so.1: python3.3: fatal: relocation error: file /opt/csw/lib/python3.3/s

[issue29518] 'for' loop not automatically breaking (index error on line of loop header)

2017-02-10 Thread Justin McNiel
Justin McNiel added the comment: While restarting didn't fix it, waiting for the next morning did, so I am afraid that that isn't possible, also, I have made tremendous changes to my code since then, but I would by happy to send you any bit of my code that you would like, it cold also

[issue29518] 'for' loop not automatically breaking (index error on line of loop header)

2017-02-09 Thread Justin McNiel
New submission from Justin McNiel: All for loops are refusing to break (Python 2.7.13) on win32 - only on execution of '.py' file an example would be:" for x in range(5): #Line 1 of main.py print x#Line 2 of main.py " and the resulting error would be:" F

[issue28639] inspect.isawaitable(native_coro) returns int

2016-11-07 Thread Justin Mayfield
New submission from Justin Mayfield: Patch available on my github fork.. https://github.com/mayfield/cpython/commit/8d50cc61f42fa280d380e9c10c420c949a619bef -- components: asyncio messages: 280280 nosy: Justin Mayfield, gvanrossum, yselivanov priority: normal severity: normal status

[issue28506] Multiprocessing Pool starmap - struct.error: 'i' format requires -2e10<=n<=2e10

2016-10-22 Thread Justin Ting
Justin Ting added the comment: Actually, on further inspection, I seem to be having a slightly different problem with the same error that I initially described now. Even after modifying my code so that each python forked off to another process was only given the following arguments: args

[issue28506] Multiprocessing Pool starmap - struct.error: 'i' format requires -2e10<=n<=2e10

2016-10-22 Thread Justin Ting
Justin Ting added the comment: Ah, should have picked that up, coding at 3:30am doesn't do wonders for keeping a clear head. Thanks Tim, I'll keep that in mind! *Justin Ting* *E* justingl...@gmail.com | *M* +61 424 751 665 | *L* *https://au.linkedin.com/in/justinyting <https://au.linkedin.

[issue28506] Multiprocessing Pool starmap - struct.error: 'i' format requires -2e10<=n<=2e10

2016-10-22 Thread Justin Ting
New submission from Justin Ting: Multiprocessing is throwing this error when dealing with large amounts of data (all floating points an integers), but none of which exceeds the number boundaries in the error that it throws: File "/root/anaconda3/lib/python3.5/multiprocessing/pool.py&q

[issue27755] Retire DynOptionMenu with a ttk Combobox

2016-10-20 Thread Justin Foo
Justin Foo added the comment: Is #24781 likely to make it into Python 3.6? Otherwise, would this patch be of any benefit in the meantime? -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/i

[issue24837] await process.wait() does not work with a new_event_loop

2016-09-04 Thread Justin Mayfield
Justin Mayfield added the comment: I agree with Guido. I spent a couple hours trying to debug some of my own code that turned out to be this issue. My use case is single threaded too. Perhaps I'm daft but I don't understand why the child watcher is part of the event loop policy. At first

[issue27755] Retire DynOptionMenu with a ttk Combobox

2016-08-15 Thread Justin Foo
Justin Foo added the comment: I wasn't sure if the ongoing work in #24781 essentially rendered my patch obsolete, so I keenly await Mark's response. Upon reflection, I think my patch is a cheap win even if it's later overhauled by other improvements. -- status: closed -> o

[issue27755] Retire DynOptionMenu with a ttk Combobox

2016-08-14 Thread Justin Foo
Justin Foo added the comment: Ah, I've noticed the folly of my ways. This sort of stuff is already being managed with patches filed under various other issues. It just wasn't obvious to me as I wasn't seeing many new improvements in the default branch or much communication on IDLE-dev

[issue22395] test_pathlib error for complex symlinks on Windows

2016-08-13 Thread Justin Foo
Justin Foo added the comment: This is no longer a problem for me and I can't reproduce why it was even a problem in the first place. -- status: open -> closed ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.or

[issue21417] Compression level for zipfile

2016-08-13 Thread Justin Foo
Justin Foo added the comment: I thought about that, but: 1. compresslevel=None might make look like no compression was being done at all 2. The current default is not 9 for all backends -- ___ Python tracker <rep...@bugs.python.org>

[issue21417] Compression level for zipfile

2016-08-13 Thread Justin Foo
Justin Foo added the comment: Would compresslevel be a suitable argument? All the backend compressors accept values from 1 to 9 if I'm not mistaken. -- nosy: +jfoo ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/i

[issue27755] Retire DynOptionMenu with a ttk Combobox

2016-08-13 Thread Justin Foo
Changes by Justin Foo <jcjf1...@gmail.com>: -- keywords: +patch Added file: http://bugs.python.org/file44097/issue27755.patch ___ Python tracker <rep...@bugs.python.org> <http://bugs.python

[issue27755] Retire DynOptionMenu with a ttk Combobox

2016-08-13 Thread Justin Foo
New submission from Justin Foo: One aspect of the IDLE interface that looks extremely old is the the dropdown menu. In the patch, I think I've preserved whatever essential functionality DynOptionMenu used to have, but I'm relatively unfamiliar with Tk so I'm not sure. -- assignee

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2016-02-10 Thread Justin Mayfield
Justin Mayfield added the comment: Alexander, That sounds unrelated. I'd treat it as a new issue until you have concrete evidence to the contrary. Also on face value it sounds like it might just be your operating systems open file limit. On OSX I think the default open file limit

[issue6766] Cannot modify dictionaries inside dictionaries using Managers from multiprocessing

2015-12-02 Thread Justin Patrin
Justin Patrin added the comment: I'm still running into these issues with Python 2.7.10. I'm trying to find a way to share dynamically allocated sub-dictionaries through multiprocessing as well as dynamically allocated RLock and Value instances. I can use the manager to create them but when I

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-19 Thread Justin Mayfield
Justin Mayfield added the comment: I should have commented more on the run_once removal. The depiction given in its docstring seemed inconsistent with the new way stop works and I found no callers, so it seemed like it was best left out to avoid confusion. No worries though, I didn't get

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-19 Thread Justin Mayfield
Justin Mayfield added the comment: Ha, email race. Regarding rev 2, the updated docstring and scheduled stop looks good along with alleviating the confusion I mentioned. I'm not sure about your warning comment; Perhaps that's a patch I didn't lay eyes on. Cheers

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-19 Thread Justin Mayfield
Justin Mayfield added the comment: I see. Seems like good discussion over there. I joined up. -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/i

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-17 Thread Justin Mayfield
Justin Mayfield added the comment: Attached patch submission for stop flag proposal. I assume you didn't mean a github PR since the dev docs seem to indicate that is for readonly usage. This passes all the tests on my osx box but it should obviously be run by a lot more folks

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-16 Thread Justin Mayfield
Justin Mayfield added the comment: You bet. -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue25593> ___ ___ Python-bugs-list

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-16 Thread Justin Mayfield
Justin Mayfield added the comment: +1 Let me know what I can do to help. -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/i

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-12 Thread Justin Mayfield
Justin Mayfield added the comment: Interesting. I was going to do an analysis what using _ready.appendleft() for adding selector events would do for that scenario. The idea being to consistently juxtapose exiting callbacks, selector events and new callbacks. However I think this just moves

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-12 Thread Justin Mayfield
Justin Mayfield added the comment: Yes, that's what I was suggesting. Looking at tornado they do the stop between callbacks/matured-scheduled and events. That approach seems somewhat arbitrary to me at first glance but tornado is very mature and they usually have good reasons for what

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-11 Thread Justin Mayfield
Justin Mayfield added the comment: I don't believe this is a case of nonidempotent callbacks, unless you are referring to Future.set_result(), which by design can't be called twice. The callbacks are given an inconsistent opportunity to modify the poll set because of indeterminacy

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-11 Thread Justin Mayfield
Justin Mayfield added the comment: Guido, Shouldn't this not be the case for level triggered polling? From looking at selectors it looks like these are always level triggered which means they should only event once. -- ___ Python tracker <

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-11 Thread Justin Mayfield
Justin Mayfield added the comment: Attached server side of repro. -- Added file: http://bugs.python.org/file41017/Issue25593_repro_server.py ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/i

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-11 Thread Justin Mayfield
Justin Mayfield added the comment: Just reproduced on Linux, Fedora Core 23. -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/i

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-11 Thread Justin Mayfield
Justin Mayfield added the comment: Attaching simplified test setup. It does take some doing to repro so the local async server is required to make it happen (for me). When I tried just pointing to python.org it would not repro in 100 iterations, but using a local dummy server repros 100

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-11 Thread Justin Mayfield
Justin Mayfield added the comment: Nevermind, in the case of writeablity it won't matter either way. -- So in looking at tornado's ioloop they run the ready callbacks before calling poll(). So the callbacks can modify the poll set. -- ___ Python

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-11 Thread Justin Mayfield
Justin Mayfield added the comment: This code repros without aiohttp when pitted against the previously attached web server (again on OSX 10.11, mid-2012 MBPr). Admittedly this may seem very arbitrary but I have better reasons in my production code for stopping an IOLoop and starting it again

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-11 Thread Justin Mayfield
Justin Mayfield added the comment: I'm attaching a patch that runs `_ready` callbacks at the start of `_run_once`. The style and implications are ranging so I leave it to you at this point. -- keywords: +patch Added file: http://bugs.python.org/file41019

[issue25593] _sock_connect_cb can be called twice resulting in InvalidStateError

2015-11-11 Thread Justin Mayfield
Justin Mayfield added the comment: I believe I'm seeing this bug in a non-threaded and non-forked env. System: OSX 10.11.1 (15B42) Python 3.5.0 (from brew install) I'm using aiohttp to create several dozens of HTTP connections to the same server (an async tornado web server). Nothing

[issue24652] C-API Pure Embedding enhancement

2015-07-17 Thread Justin Huang
New submission from Justin Huang: From the example in here: https://docs.python.org/2/extending/embedding.html#pure-embedding when directly using the example (compiling and trying with external file etc.) it doesn't work right away. Instead an extra line: PySys_SetArgv(argc, argv

[issue24645] logging.handlers.QueueHandler should not lock when handling a record

2015-07-16 Thread Justin Bronder
Justin Bronder added the comment: On 16/07/15 20:03 +, R. David Murray wrote: R. David Murray added the comment: Can you expand on the deadlock? Are you saying that the extra locking is causing the deadlock? -- nosy: +r.david.murray, vinay.sajip versions: -Python 3.2

[issue24645] logging.handlers.QueueHandler should not lock when handling a record

2015-07-16 Thread Justin Bronder
New submission from Justin Bronder: The Queue backing the QueueHandler is already sufficiently locking for thread-safety. This isn't a huge issue, but the QueueHandler is a very nice built-in which could be used to work around a deadlock I've encountered several times. In brief, functions

[issue24645] logging.handlers.QueueHandler should not lock when handling a record

2015-07-16 Thread Justin Bronder
Justin Bronder added the comment: On 16/07/15 23:21 +, Vinay Sajip wrote: Vinay Sajip added the comment: I'm not sure I want to make a special case just to support what seems like a somewhat pathological use case (no offence intended). If you need this, there's no reason you

[issue23429] -5**4 returns -625 instead of 625

2015-02-10 Thread Justin
Justin added the comment: I think there is a misunderstanding of precedence here. If we had an operation were -1 * 5**4, I could fully understand the statement of precedence. However, in the absence of the -1 *, the -5 becomes a single atom. For example: myVal = -5 myVal**4 625 # not -625

[issue23429] -5**4 returns -625 instead of 625

2015-02-10 Thread Justin
Justin added the comment: Thank you for the link. I can understand the language definition and I can accept it at face value. The confusion I have is that it does not appear to be consistent. My example of -5**4 vs 4**-2 would both have literals with a unary sign. One works as expected

[issue23429] -5**4 returns -625 instead of 625

2015-02-09 Thread Justin
New submission from Justin: C:\Users\Justinpython Python 3.4.2 (v3.4.2:ab2c023a9432, Oct 6 2014, 22:15:05) [MSC v.1600 32 bit (Intel)] on win32 Type help, copyright, credits or license for more information. print(-5**4) -625 #note...this should be 625 and not a negative 625

[issue23317] Incorrect description of descriptor invocation in Python Language Reference

2015-01-26 Thread Justin Eldridge
Justin Eldridge added the comment: Ah, I see how writing a description of this which is both concise and precise would be difficult, especially for Python 2. But the Class Binding description is correct, since if the __get__ method is *not* defined on the type (of the descriptor

[issue23317] Incorrect description of descriptor invocation in Python Language Reference

2015-01-25 Thread Justin Eldridge
New submission from Justin Eldridge: The section titled Invoking Descriptors in the Python Language Reference [1] says: Class Binding If binding to a new-style class, A.x is transformed into the call: A.__dict__['x'].__get__(None, A). This suggests that __get__ is looked

[issue22395] test_pathlib error for complex symlinks on Windows

2014-09-13 Thread Justin Foo
Justin Foo added the comment: The failing tests were: * test_complex_symlinks_absolute * test_complex_symlinks_relative * test_complex_symlinks_relative_dot_dot for both PathTest and WindowsPathTest, via inheritance from the _BasePathTest class

[issue22395] test_pathlib error for complex symlinks on Windows

2014-09-12 Thread Justin Foo
New submission from Justin Foo: The _check_complex_symlinks function compares paths for string equality instead of using the assertSame helper function. Patch attached. -- components: Tests messages: 226828 nosy: jfoo priority: normal severity: normal status: open title: test_pathlib

[issue22395] test_pathlib error for complex symlinks on Windows

2014-09-12 Thread Justin Foo
Justin Foo added the comment: The _check_complex_symlinks function compares stringified paths for string equality instead of using the assertSame helper method. Patch attached. -- keywords: +patch Added file: http://bugs.python.org/file36609/issue22395.patch

[issue21699] Windows Python 3.4.1 pyvenv doesn't work in directories with spaces.

2014-06-09 Thread Justin Engel
New submission from Justin Engel: Venv virtual environments don't work with spaces in the path. (env) C:\My Directory\path pip -V Fatal error in launcher: Unable to create process using 'C:\My Directory\path\env\Scripts\python.exe C:\My Directory\path\env\Scripts\pip.exe -V

[issue20849] add exist_ok to shutil.copytree

2014-04-23 Thread Justin Myers
Changes by Justin Myers jus...@justinmyers.net: -- nosy: +justin.myers ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue20849 ___ ___ Python-bugs

[issue13936] RFE: change bool(datetime.time(0, 0, 0)) to evaluate as True

2014-03-13 Thread Justin Brown
Justin Brown added the comment: This behavior conflicts with the other major classes, datetime.date and datetime.datetime. The ostensible reason for this falsy behavior is that midnight represents a fundamental zero point. We should expect to see similar zero points that evaluate to False

[issue7511] msvc9compiler.py: ValueError when trying to compile with VC Express

2014-02-25 Thread Justin Foo
Justin Foo added the comment: I think getting this bug fixed would be really nice from a user experience point of view. I've further into this for my own setup, which is Python 3.3 64-bit, Visual C++ Express 2010 and the Windows SDK v7.1 (plus service packs), which is probably a typical

[issue20690] IDLE Indents convert to spaces and then throws error

2014-02-19 Thread Justin Barker
New submission from Justin Barker: When I add some lines to my module code, some of the tabs that I enter convert to spaces. This throws an inconsistency error and I can't get it to stop! -- components: IDLE files: test.py messages: 211664 nosy: Justin.Barker priority: normal severity

[issue20301] Correct docs for default access argument for DeleteKeyEx

2014-01-19 Thread Justin Foo
New submission from Justin Foo: The default access for winreg.DeleteKeyEx is winreg.KEY_WOW64_64KEY (as per the function signature). This the documentation for Python 2.7 has this correct. Reference: http://hg.python.org/cpython/file/2e32462e4832/PC/winreg.c#l1089 -- assignee: docs

  1   2   3   >