Andrei Kulakov added the comment:
Related (with PR up): https://bugs.python.org/issue42861
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue42
New submission from Andrei Kulakov :
breakpoint 5 does not exist:
(Pdb) commands 5
(com) p x
(com)
(com) end
--
components: Library (Lib)
messages: 397852
nosy: andrei.avk
priority: normal
severity: normal
status: open
title: Pdb commands allows to add commands to invalid breakpoint
Change by Andrei Kulakov :
--
keywords: +patch
pull_requests: +25799
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27252
___
Python tracker
<https://bugs.python.org/issu
Change by Andrei Kulakov :
--
keywords: +patch
nosy: +andrei.avk
nosy_count: 4.0 -> 5.0
pull_requests: +25809
stage: test needed -> patch review
pull_request: https://github.com/python/cpython/pull/27265
___
Python tracker
<https://bugs.p
Andrei Kulakov added the comment:
I've put up a simple PoC PR adding a __field_doc__ optional dict attr to
dataclass, which would add the docs to class docstring:
@dataclass
class A:
__field_doc__ = dict(num='number of widgets', total='total widgets')
to
Andrei Kulakov added the comment:
John-Mark: each fields doc can be added to the dictionary separately
though..
--
___
Python tracker
<https://bugs.python.org/issue42
Andrei Kulakov added the comment:
Thanks for reporting Jakub, and for patching Gregory!
--
___
Python tracker
<https://bugs.python.org/issue43219>
___
___
Pytho
Andrei Kulakov added the comment:
John-Mark: yep, it's just a draft patch for now.
It doesn't need to be two lines; it can be updated to start with an empty dict
and perhaps use a shorter special name for the doc dict and then you can do:
f1:int = 1
FDOC['f1'] = 'fo
Andrei Kulakov added the comment:
Eric: makes sense, I'll go with that.
John-Mark: I will go ahead and work on a PR along the lines Terry suggested,
let us know if you have any objections or if you would prefer to work on a PR
by yourself instead.
P.S. I understand now what you meant
Change by Andrei Kulakov :
--
pull_requests: +25824
pull_request: https://github.com/python/cpython/pull/27279
___
Python tracker
<https://bugs.python.org/issue42
Andrei Kulakov added the comment:
I've added a new PR: https://github.com/python/cpython/pull/27279
(note it's a rough PoC draft at this point)
The output of inspect.getdoc() is the same:
A(total: int, num: int = 5)
total: int -- number of widgets
num: int [5] -- tot
Andrei Kulakov added the comment:
I haven't modified `_finddoc` in my PR because it currently doesn't show all
existing dataclass fields (only those with default set) -- therefore it would
make sense to consider this addition if / when complete set of dataclass fields
is added t
Change by Andrei Kulakov :
--
keywords: +patch
nosy: +andrei.avk
nosy_count: 1.0 -> 2.0
pull_requests: +25855
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27310
___
Python tracker
<https://bugs.python.org/i
Andrei Kulakov added the comment:
I've put up a PR here: https://github.com/python/cpython/pull/27310/files
If this looks good, I will update the docs and add news entry.
--
___
Python tracker
<https://bugs.python.org/is
Andrei Kulakov added the comment:
I was thinking about adding this to named tuple as well, but it's less useful
than with dataclasses because named tuples are simpler and smaller. However if
this feature is added to dataclasses and is widely used, it would make a lot of
sense to expa
Andrei Kulakov added the comment:
This can be closed (fixed).
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue42357>
___
___
Python-bug
Andrei Kulakov added the comment:
3.6 no longer gets bugfixes, so I think this can be closed.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue26
Andrei Kulakov added the comment:
I have confirmed the reproducer posted by Ronald also works for me, on 3.11
compiled from dev branch, also with cpu count == 8; on the new M1 silicon (that
shouldn't matter though).
I agree this can be closed.
--
nosy: +andrei.avk
status: pe
Andrei Kulakov added the comment:
I agree not backporting is most likely fine; just want to add that the issue
here is that it allows you to enter command for an invalid breakpoint, creating
a strong impression both that bpoint is valid and that command will be active
Andrei Kulakov added the comment:
The same thing happens in 3.11. I've also confirmed the markup is exactly the
same.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/is
Andrei Kulakov added the comment:
In this case, it appears that ref with tilde and object makes it work in both
3.10 and 3.11, but with just bare method specified, it doesn't create link in
both -- so it may be good to try updating it to have tilde and a ref to obj:
Once an itera
Andrei Kulakov added the comment:
Something changed between 3.8 and 3.9 so that "bare" references to methods on
`object` no longer get linked. So for example, :meth:`__bool__` would get link
to object.__bool__ anchor, but in 3.9+, it should be :meth:`~object.__bool__`
for the
Change by Andrei Kulakov :
--
type: -> behavior
versions: +Python 3.11, Python 3.9
___
Python tracker
<https://bugs.python.org/issue42182>
___
___
Python-
Change by Andrei Kulakov :
--
nosy: +andrei.avk
nosy_count: 13.0 -> 14.0
pull_requests: +25916
pull_request: https://github.com/python/cpython/pull/27384
___
Python tracker
<https://bugs.python.org/issu
Andrei Kulakov added the comment:
I put the PR up here:
https://github.com/python/cpython/pull/27384/files
--
___
Python tracker
<https://bugs.python.org/issue42
Change by Andrei Kulakov :
--
keywords: +patch
pull_requests: +25917
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27384
___
Python tracker
<https://bugs.python.org/issu
Andrei Kulakov added the comment:
Can be closed as a dupe of https://bugs.python.org/issue42958, which has a PR
ready for review.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue33
Andrei Kulakov added the comment:
Note that SETUP_WITH doc was completely removed. Not sure if that was
intentional (since the opcode itself still exists).
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue42
Andrei Kulakov added the comment:
Aviram: see issue https://bugs.python.org/issue17340 , there is a lot of old
discussions along the same lines.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue40
Andrei Kulakov added the comment:
Françoise:
You can simply set the two attrs required by __repr__ in the pdb session:
./python.exe ~/temp/a.py
--INS--
> /Users/ak/opensource/cpython/Lib/logging/__init__.py(1152)__ini
Andrei Kulakov added the comment:
I was able to reproduce something very similar, and I believe essentially the
same issue perhaps varying due to config. I think I've actually seen similar
glitches with python command line readline handling of k.
So, entering 1234 always works fine
Andrei Kulakov added the comment:
Also this bug happens on both iterm2 and terminal on MacOS, but note that on
'terminal', it's impossible to ctrl-c or ctrl-z out of the test script, so if
you test it, you will need to close the window or the tab, and lose shell
history and
Andrei Kulakov added the comment:
Joakim: by the way, what I was able to reproduce is just a visual bug. IOW, the
text is still the same and all there in the buffer, but it shows up only after
a combination of rightward movement and 'a', and a copy of it shows up on the
left sid
Change by Andrei Kulakov :
--
keywords: +patch
nosy: +andrei.avk
nosy_count: 3.0 -> 4.0
pull_requests: +25989
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27470
___
Python tracker
<https://bugs.python.org/i
Andrei Kulakov added the comment:
I've put up a PR here: https://github.com/python/cpython/pull/27470/files
--
___
Python tracker
<https://bugs.python.org/is
Change by Andrei Kulakov :
--
nosy: +andrei.avk
nosy_count: 3.0 -> 4.0
pull_requests: +25990
pull_request: https://github.com/python/cpython/pull/27471
___
Python tracker
<https://bugs.python.org/issu
Andrei Kulakov added the comment:
I've put up a PR that expands the docs for NotADirectoryError here:
https://github.com/python/cpython/pull/27471/files
--
___
Python tracker
<https://bugs.python.org/is
Andrei Kulakov added the comment:
Danny: that would be inconsistent with the rest of the doc, it's written as a
reference list for exceptions and none of them have examples currently.. It
also seems fairly clear without an example.
--
___
P
Andrei Kulakov added the comment:
(in fact, there is one example (for f-string exception) but the overall pattern
is to just have reference type descriptions)
--
___
Python tracker
<https://bugs.python.org/issue41
Andrei Kulakov added the comment:
Danny: then it would be probably more useful to say "On POSIX systems, it will
be raised when ...".
I'm ambivalent on whether this is needed.
Eryk: wdyt?
--
___
Python tracker
<https
Andrei Kulakov added the comment:
Łukasz: I agree.
--
___
Python tracker
<https://bugs.python.org/issue43625>
___
___
Python-bugs-list mailing list
Unsubscribe:
Andrei Kulakov added the comment:
Eryk: sounds good, updated the PR.
--
___
Python tracker
<https://bugs.python.org/issue41737>
___
___
Python-bugs-list mailin
Andrei Kulakov added the comment:
Eryk: On MacOS, I get Not a directory error on all 3:
NotADirectoryError: [Errno 20] Not a directory: 'spam'
NotADirectoryError: [Errno 20] Not a directory: 'spam/'
NotADirectoryError: [Errno 20] Not a d
Andrei Kulakov added the comment:
This can be closed as fixed.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue19270>
___
___
Python-bug
Andrei Kulakov added the comment:
this can be closed I believe.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue42026>
___
___
Python-bug
Change by Andrei Kulakov :
--
keywords: +patch
nosy: +andrei.avk
nosy_count: 3.0 -> 4.0
pull_requests: +26034
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27519
___
Python tracker
<https://bugs.python.org/i
Andrei Kulakov added the comment:
I've put up the PR here that makes availability clearer in the docs:
https://github.com/python/cpython/pull/27519/files
--
___
Python tracker
<https://bugs.python.org/is
Change by Andrei Kulakov :
--
type: -> behavior
versions: +Python 3.10, Python 3.11, Python 3.9
___
Python tracker
<https://bugs.python.org/issue41983>
___
_
Andrei Kulakov added the comment:
This was fixed in https://github.com/python/cpython/pull/26300
so the issue can be closed as fixed, and the PR can also be closed.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue42
Andrei Kulakov added the comment:
dubiousjim: I agree, it should be `and` or have the form `not (isinstance(..)
or hasattr(..))`.
If you would like to make the patch, I can review.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.
Andrei Kulakov added the comment:
By the way I confirmed that fixing this doesn't break any tests (perhaps
unsurprising ;) )
--
___
Python tracker
<https://bugs.python.org/is
Andrei Kulakov added the comment:
I haven't looked too much into typing module, so take this with a grain of
salt, but my understanding is that TypedDict is a special construct (as the
first sentence in the docs defines it), which can be used as a base. I think
most users would con
Andrei Kulakov added the comment:
I think this can be closed.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue42019>
___
___
Python-bug
Change by Andrei Kulakov :
--
keywords: +patch
nosy: +andrei.avk
nosy_count: 2.0 -> 3.0
pull_requests: +26044
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27529
___
Python tracker
<https://bugs.python.org/i
Andrei Kulakov added the comment:
I agree that the doc is wrong; I've put up the tiny PR here:
https://github.com/python/cpython/pull/27529/files
Note that linecache.getline() returns line with \n but is then .strip()'ed.
--
___
Pyth
Andrei Kulakov added the comment:
Yannick: the PRs are submitted via github; you can find more detail on
contributions workflow / process here:
https://devguide.python.org/
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.
Andrei Kulakov added the comment:
This issue is due to *drop_whitespace* arg being True by default.
Documentation states that *drop_whitespace* applies after wrapping, so the fix
in the PR would break that promise because, as far as I understand, it applies
the wrapping after
Andrei Kulakov added the comment:
Pablo: thanks for investigating and fixing this, I will look for examples of
changing or patching sys.modules in the testsuite to see how to avoid global
effects. I think we can keep this issue open for the follow-up discussion / fix?
--
nosy
Andrei Kulakov added the comment:
I've been looking into avoiding global modification of modules['__main__'], the
options are:
1. remove the test and leave it at that
2. remove the test and open a new issue for re-adding a test ; and look for a
way to do it safely
3
Change by Andrei Kulakov :
--
pull_requests: +26078
pull_request: https://github.com/python/cpython/pull/27571
___
Python tracker
<https://bugs.python.org/issue44
Andrei Kulakov added the comment:
Please disregard my last comment, I think I found a good way to fix it, the PR
is up here:
https://github.com/python/cpython/pull/27571/files
--
___
Python tracker
<https://bugs.python.org/issue44
Andrei Kulakov added the comment:
I think fix to make `drop_whitespace=False` stable, can be as simple as adding
two lines in `_munge_whitespace()`:
+text = re.sub(r' \n', ' ', text)
+text = re.sub(r'\n ', ' ',
Andrei Kulakov added the comment:
Irit: I assume you mean r' \r?\n', that's a great idea, it's much faster than
adding a separate replacement step.
Latest version I came up with is this:
if re.search(r' \r?\n', text):
text
Andrei Kulakov added the comment:
Note that I'm not handling a single '\r' because that was before Mac OS X; but
it is handled by the following line (i.e. by the old logic):
text = text.translate(self.unicode_w
Change by Andrei Kulakov :
--
pull_requests: +26089
pull_request: https://github.com/python/cpython/pull/27587
___
Python tracker
<https://bugs.python.org/issue32
Andrei Kulakov added the comment:
I've added an initial draft PR:
https://github.com/python/cpython/pull/27587/files
I will add docs and news if this looks good in general.
--
___
Python tracker
<https://bugs.python.org/is
Andrei Kulakov added the comment:
Some examples were added since this issue was created, but more can be added.
Stargirl: feel free to open a PR, I can review it.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue41
Andrei Kulakov added the comment:
This was fixed in https://github.com/python/cpython/pull/25786 , and so can be
closed.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue36
Andrei Kulakov added the comment:
Jack: I'm not sure if this eventual fix is worth doing.. it might cause issues
for users to leave it as is, but can also cause issues to remove it even after
deprecation period; I don't have a good feeling if one is more likely than the
other,
Andrei Kulakov added the comment:
It looks like all the pages listed were updated with working links to current
stable release docs, so this can be closed and if somebody finds more broken
links, this issue can be reopened later.
--
nosy: +andrei.avk
Andrei Kulakov added the comment:
This was fixed in https://github.com/python/cpython/pull/26300 but wasn't
backported.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/is
Andrei Kulakov added the comment:
I believe this can be closed now..
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue41779>
___
___
Pytho
Andrei Kulakov added the comment:
This was fixed so can be closed.
Current datetime docs have a footnote for %Z:
...
strptime() only accepts certain values for %Z:
any value in time.tzname for your machine’s locale
the hard-coded values UTC and GMT
--
nosy: +andrei.avk
Andrei Kulakov added the comment:
+1 for '... and uses a simple but effective approach ..'
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.o
New submission from Andrei Kulakov :
(note the actual relevant code is in posixmodule.c)
os.stat() error can be confusing and misleading when given an fd and with
follow_symlinks=False:
ValueError: stat: cannot use fd and follow_symlinks together
It's less bad when os.stat() is
Andrei Kulakov added the comment:
> In addition, fwalk() can probably check that `top` arg is a string and raise
> an error stating that it should be a string if it's not.
I should clarify that fwalk() cannot accept an fd as the `top` arg, see
https://bugs.python.org
Change by Andrei Kulakov :
--
pull_requests: +26159
pull_request: https://github.com/python/cpython/pull/27671
___
Python tracker
<https://bugs.python.org/issue44
New submission from Andrei Zene :
In an application where we were handling incoming requests in new threads, we
noticed that the memory usage grew over time. After trying to understand what's
going on, i was able to reproduce this with a smaller python script that i've
attached.
New submission from Andrei Kulakov :
RFC 4648 [1] has added a list of encoding test cases -- see section 10 of the
RFC. It might be nice to add a test function that is a direct copy of these
test cases. This will make conformance to this RFC clearer (actually right now
we don't
Change by Andrei Kulakov :
--
keywords: +patch
nosy: +andrei.avk
nosy_count: 4.0 -> 5.0
pull_requests: +26186
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27700
___
Python tracker
<https://bugs.python.org/i
Andrei Kulakov added the comment:
Unfortunately modern laptop keyboards have almost no key travel and barely any
tactile feedback [*]. Users on such keyboards really do need feedback for each
key pressed. Not providing an option for such feedback is in effect forcing
users to choose
Andrei Kulakov added the comment:
Also see https://bugs.python.org/issue42255
Hugo: can you convert the patch to a PR?
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue24
Change by Andrei Kulakov :
--
keywords: +patch
pull_requests: +26225
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27747
___
Python tracker
<https://bugs.python.org/issu
Change by Andrei Kulakov :
--
keywords: +patch
nosy: +andrei.avk
nosy_count: 8.0 -> 9.0
pull_requests: +26226
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27748
___
Python tracker
<https://bugs.python.org/i
Change by Andrei Kulakov :
--
keywords: +patch
nosy: +andrei.avk
nosy_count: 8.0 -> 9.0
pull_requests: +26227
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27750
___
Python tracker
<https://bugs.python.org/i
Andrei Kulakov added the comment:
This can be closed; from the PR note:
Resolved in PR #21624, can close.
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue41
Andrei Kulakov added the comment:
How about returning the updated dicts from init() and showing an example in the
docs of such usage?
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue34
Andrei Kulakov added the comment:
Terry: thanks! - that's good to know.
--
___
Python tracker
<https://bugs.python.org/issue41234>
___
___
Python-bugs-l
Andrei Kulakov added the comment:
It seems like the documentation is lacking and perhaps misleading in regard to
attributes.
- anything hashable can be used as a key in an obj.__dict__: an int, a tuple,
etc. According to __dict__ docs, all of those are attributes. There's no
warning
Andrei Kulakov added the comment:
In the last message I've said that according to __dict__ docs, anything in
__dict__ is an attribute of respective obj. That's a bit too-strongly worded,
the docs can be understood in the sense that anything that ends up in __dict__
via other
Andrei Kulakov added the comment:
Xiaolong: The file no longer exists on the google drive. How big was the file?
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue41
Andrei Kulakov added the comment:
In current dev version (3.11), _PosixFlavor doesn't have gethomedir(), and
instead os.path.expanduser() is used via _NormalAccessor(). So at least for the
current dev version, the docs are accurate.
--
nosy: +andre
Andrei Kulakov added the comment:
This issue was fixed by https://github.com/python/cpython/pull/26903
(I confirmed that the error no longer happens with attached file).
This can be closed as fixed.
--
nosy: +andrei.avk
___
Python tracker
<ht
Andrei Kulakov added the comment:
I will add the test in the next 1-2 days, I should have asked if it's needed in
this case..
--
___
Python tracker
<https://bugs.python.org/is
Change by Andrei Kulakov :
--
pull_requests: +26308
stage: test needed -> patch review
pull_request: https://github.com/python/cpython/pull/27846
___
Python tracker
<https://bugs.python.org/issu
Andrei Kulakov added the comment:
I wrote a longer explanation but BPO and Chrome ate it.
The issue is that DoubleVar is not locale aware. I don't know what should be
the actual fix but the following may be a useful workaround (It works on my
system):
import tkinter
import locale
i
Andrei Kulakov added the comment:
That test does have a yield statement in it, and does cause the deprecation
warning to fire off. I'm looking into it..
--
___
Python tracker
<https://bugs.python.org/is
Andrei Kulakov added the comment:
Yes, it seems like this deprecation detected a broken test that wasn't running.
If I add a `raise` statement to it and rerun the test, it still passes on my
local system:
./python.exe -m unitte
Andrei Kulakov added the comment:
The test was refactored in this commit:
https://github.com/python/cpython/commit/658103f84ea860888f8dab9615281ea64fee31b9
the refactoring moved yield statement from inner func into the test.
That was 8 years ago so it hasn't worked since then, and
Change by Andrei Kulakov :
--
nosy: +andrei.avk
___
Python tracker
<https://bugs.python.org/issue44968>
___
___
Python-bugs-list mailing list
Unsubscribe:
201 - 300 of 528 matches
Mail list logo