Terry J. Reedy added the comment:
New changeset 09a5c077bd8d444e9701b4d1314160b8904434e0 by Terry Jan Reedy in
branch '3.6':
[3.6] bpo-30928: Update idlelib/NEWS.txt. (GH-6995) (GH-6997)
https://github.com/python/cpython/commit/09a5c077bd8d444e9701b4d1314160
Terry J. Reedy added the comment:
The patch is trivial, and I assume that the test change is adequate, but I want
to put this issue on hold for the present because
a) the performance gain is minuscule (the code cleanup is worth a bit more);
b) I am reluctant to backport something to 3.6 that
Terry J. Reedy added the comment:
I should mention that using Treeview could mean not using pyclbr.Instead,
we might extract the parsing loop and modify it to build tree nodes as
functions and classes are encountered.
--
___
Python tracker
Terry J. Reedy added the comment:
The patch seems to have worked. The last AppVeyor failure was a day ago, when
testing the 3.7 backport of the fix.
https://ci.appveyor.com/project/python/cpython/history
--
___
Python tracker
<ht
Terry J. Reedy added the comment:
There is a reference to this issue in #31494, msg302628 (CodeContext loops).
--
___
Python tracker
<https://bugs.python.org/issue31
New submission from Terry J. Reedy :
There are a number of possible improvements to CodeContext. They be separate
issues (and PRs) that are dependencies of this master index issue. Some should
be fairly easy now that we have the new tests.
1. #32831 added docstrings and tests. Review has
Change by Terry J. Reedy :
--
dependencies: +IDLE cond context: fix code update and font update timers, IDLE:
Add docstrings and tests for codecontext, Idle Code Context: separate changing
current and future editors
stage: test needed ->
title: IDLE: Improve CodeContext (vari
Terry J. Reedy added the comment:
If you are following up with a new patch that includes code changes, make a new
issue. In the opening message, please briefly list the mini-issues fixed so I
can refer changes to purposes. Perhaps something like
* getspacesfirstword - function and param1
Terry J. Reedy added the comment:
The patch fixed immediate problem #2 above. #1 is a separate enhancement and I
listed it as #4 on the new master issue #33610.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -&g
Terry J. Reedy added the comment:
11. Use read-only Text instead of Label for context.
Text.index(f'@{e.x},{e.y}', where e is event, converts mouse click to line
number. That can be mapped to text line number for 8. above. Text also enables
12. If context box is too short for
Terry J. Reedy added the comment:
Super list. It made the diff really easy to parse into related changes. I
printed it, reviewed code for items one by one, made a few changes, and checked
them off when satisfied.
IDLE *does* (sort of) allow tabs. One can read a file with tabs or Format
Terry J. Reedy added the comment:
New changeset 8506016f904ba6bc27bf5261f27a0bdd5945cd26 by Terry Jan Reedy
(Cheryl Sabella) in branch 'master':
bpo-33628: IDLE: Minor code cleanup of codecontext.py and its tests (GH-7085)
https://github.com/python/cpyt
Change by Terry J. Reedy :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Terry J. Reedy added the comment:
Point 1 is #33397.
--
___
Python tracker
<https://bugs.python.org/issue33396>
___
___
Python-bugs-list mailing list
Unsub
Terry J. Reedy added the comment:
With #33628 merged, I consider the next priorities to be 5. #22703 (overt bug,
highest), 7, and 8, to make the feature more usable.
10. is a vague speculative feature that would use get_indent_firstword but
would actually be for the editor. Update problem
Terry J. Reedy added the comment:
[In #33610, msg317601, Cheryl Sabella wrote, copying here:]
I've started looking at #22703, more specifically, I've been trying to recreate
it. Since the config changes in 3.6/3.7 where the flag was removed from config
dialog, the code context n
Terry J. Reedy added the comment:
Cheryl, you are right. The extension-feature conversion made this issue mostly
obsolete. Editors start without code context and the menu only toggles the
current window.
If I had thought about it, I would have closed this or added an update last
fall
Terry J. Reedy added the comment:
An experienced non-CS computer user I asked does not like 'toggle x' or 'switch
x', but would like whichever of 'show x' and 'hide x' is appropriate (rather
than both). For breakpoints, this would mean one menu e
Terry J. Reedy added the comment:
Changing title since 'separation' has already been done. Adding a user
configuration option is a separate issue.
--
title: Idle Code Context: separate changing current and future editors -> Idle
Code Context
Terry J. Reedy added the comment:
2 quick thoughts.
1. I am not sure we can/should change the default, as it interacts with the
recorded user changes for past versions. The default is that we never change
defaults. (Having one user config for all Python/IDLE versions is a curse as
well as
Terry J. Reedy added the comment:
I agree.
--
assignee: -> docs@python
components: +Documentation
nosy: +docs@python, terry.reedy
resolution: not a bug ->
stage: -> needs patch
title: statistics.median does not work with ordinal scale -> statistics.median
does not work
Change by Terry J. Reedy :
--
nosy: +yselivanov
versions: -Python 3.5
___
Python tracker
<https://bugs.python.org/issue33582>
___
___
Python-bugs-list mailin
Change by Terry J. Reedy :
--
stage: -> patch review
type: -> performance
versions: +Python 3.8
___
Python tracker
<https://bugs.python.org/issue33587>
___
__
Change by Terry J. Reedy :
--
nosy: +yselivanov
title: inspect.getsource performs unnecessary filesystem stat call which is
ignored in some circumstances -> inspect.getsource performs unnecessary
filesystem stat call
___
Python tracker
<
Terry J. Reedy added the comment:
See also #33582 for 'formatargspec'.
--
nosy: +terry.reedy, yselivanov
versions: -Python 3.5
___
Python tracker
<https://bugs.python.o
Change by Terry J. Reedy :
--
pull_requests: +6757
___
Python tracker
<https://bugs.python.org/issue33595>
___
___
Python-bugs-list mailing list
Unsubscribe:
Terry J. Reedy added the comment:
New changeset 804fcf66559992db9d23695e501c502ab20b7712 by Terry Jan Reedy in
branch '2.7':
[2.7] bpo-33595: Fix lambda parameters being refered as arguments (GH-7037)
(GH-7122)
https://github.com/python/cpython/commit/804fcf66559992db9d23695e501c50
Terry J. Reedy added the comment:
Thanks for the fix.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
title: FIx references to lambda "arguments" -> Fix references to lambda
"arguments"
__
Terry J. Reedy added the comment:
Your proposal is underspecified. (And 'yours' is inconsistent with 'your';-).
If you want sequential replacememt in multiple scans, make sequential replace
calls. Use a loop if the number of replacement scans is variable or large. To
Terry J. Reedy added the comment:
Can you check the copyright and license of the recipe text and code?
--
nosy: +terry.reedy
___
Python tracker
<https://bugs.python.org/issue33
Terry J. Reedy added the comment:
The Python Cookbook, based on the site, says that copyright of each recipe is
retained by original authors. If so authors who have signed the PSF CLA can
contribute their own work.
Since Active State must have collectively licensed the recipes to O'R
Terry J. Reedy added the comment:
I have not seen test_asyncio fail on AppVeyor since about Sunday.
https://ci.appveyor.com/project/python/cpython/history
shows only a couple of failures, which could very well be real.
I have seen test_asyncio fail on Travis since Sunday.
With my local
Terry J. Reedy added the comment:
I presume the above is using the month-old openssl-bin-1.1.0h.
--
___
Python tracker
<https://bugs.python.org/issue32
New submission from Terry J. Reedy :
On IDLE-dev today, Elisha Paine, from Ranelagh School in England, wrote:
'''
I love IDLE (so simple and lightweight compared with other IDEs) and was just
wondering if you could add the following code into pyshell.py (as I have done
on mine)
Change by Terry J. Reedy :
--
keywords: +patch
pull_requests: +6771
___
Python tracker
<https://bugs.python.org/issue33656>
___
___
Python-bugs-list mailin
Terry J. Reedy added the comment:
I changed the bare except: to 'except AttributeError:'. If
ctypes.windll.shcore.SetProcessDpiAwareness exists, can calling it with True
ever fail?
--
___
Python tracker
<https://bugs.python.o
Change by Terry J. Reedy :
--
pull_requests: -6745
___
Python tracker
<https://bugs.python.org/issue33642>
___
___
Python-bugs-list mailing list
Unsubscribe:
Terry J. Reedy added the comment:
Unlinked duplicate PR entry. I have no idea how that could happen.
--
___
Python tracker
<https://bugs.python.org/issue33
Terry J. Reedy added the comment:
'deleted somehow': Christian added himself while I had this page open, so when
I submitted without refreshing, the nosy list did not include him. When I saw
the red warning, I did not notice the nosy list change; I s
New submission from Terry J. Reedy :
When tk and hence, IDLE, opens a text window, it contains an integral number of
lines of text in the base font. Clicking a vertical scrollbar arrow button
moves text up or down exactly one line. Clicking in the trough between the
buttons and and the
Terry J. Reedy added the comment:
This is 'out of date' in the sense that I have decided that the root problem is
scrolling by pixels instead of lines. Once we do the latter, for #33664, we
can just scroll by, say, 5 lines instead of 3, which I think is sufficient.
--
Terry J. Reedy added the comment:
After testing the patch as a user, I would like 3 user-visible changes.
1. Only display a blank line when there are no context lines.
2. Replace the 'lines' option with a 'maxlines' option. There is no need to
worry about, or explain t
Terry J. Reedy added the comment:
While trying out variable lines, #33642, I encounter two related issues.
First is the context colors. Black on gray is a variant of black on white. I
am using the dark theme, white on dark blue. For fixed sized context, the high
contrast is okay. For
Terry J. Reedy added the comment:
Thanks for the comments:
Ronald: I agree that an int should be passed. I understand your ctype comment
as saying that an attribute error before making the call should be the only
possible exception.
I should have explained the 'unrelated change&#x
Terry J. Reedy added the comment:
Eryk Sun commented on the PR.
--
___
Python tracker
<https://bugs.python.org/issue33656>
___
___
Python-bugs-list mailin
Terry J. Reedy added the comment:
The release candidate has been re-scheduled for June 11. So we should be able
to merge the minimal upgrade before that and get it in 3.7.0 and 3.6.6. (The
latter will come out at the same time as the former.)
Variable lines is about ready. Colors looks
Terry J. Reedy added the comment:
On Windows, I observe the following behaviors.
1. When Code Context is enabled on a windows still at its initial height,
context lines are added to code lines to increase the window height.
2. When the window height is changed with a mouse, that becomes the
Terry J. Reedy added the comment:
5, reformulated. Now that 'Code Context' on the Options menu only toggles a
feature for the current window, like 'Zoom Height' on the Window menu, both
should appear together on the same menu. My current inclination is to move
Terry J. Reedy added the comment:
18. Error or bounds checking for maxlines entry. I believe this really a
config dialog issue that applies to everything, but maxlines should be at least
1, with some reasonable max. In the meanwhile, we could check whether crazy
entries (text, negatives
Terry J. Reedy added the comment:
19. idle.rst doc change
20. What's New for 3.6.6 and 3.7.0
These could be patches on this issue.
21. idlelib/NEWS.txt entries
--
___
Python tracker
<https://bugs.python.org/is
Change by Terry J. Reedy :
--
stage: -> test needed
versions: +Python 3.8 -Python 3.4, Python 3.5
___
Python tracker
<https://bugs.python.org/issu
Terry J. Reedy added the comment:
Please post the code you used to test this.
--
nosy: +serhiy.storchaka, terry.reedy
stage: -> needs patch
versions: +Python 3.7, Python 3.8
___
Python tracker
<https://bugs.python.org/issu
Terry J. Reedy added the comment:
I ran py -3.7 -m test -j11 -uall
I saw test_marshal fail with fatal exception.
I saw test_tools and test_site fail as below.
C:\Users\Terry>py -3.7 -X utf8 -m test test_pkg
0:00:00 [1/1] test_pkg
1 test OK.
test_tools failed -- running: test_io (103
Terry J. Reedy added the comment:
New changeset 29996a1c4e8bd6dde6adce2b44d11a0982a47a3a by Terry Jan Reedy
(Cheryl Sabella) in branch 'master':
bpo-33642: IDLE: Use variable number of lines in CodeContext. (GH-7106)
https://github.com/python/cpyt
Terry J. Reedy added the comment:
I merged 'variable lines' and edited 'colors' to fix the resulting conflicts.
Travis passed the latter so I expect to merge it also. To avoid more
conflicts, you should wait to create the 'alignment' branch until 'colo
Terry J. Reedy added the comment:
I am leaving this open until the new behavior is documented, even though it
might be part of another issue that includes other context changes.
For this issue, I will leave what happens when unzooming unspecific
Terry J. Reedy added the comment:
New changeset de6516264e793be991f692fdd892707afb9104a7 by Terry Jan Reedy
(Cheryl Sabella) in branch 'master':
bpo-33679: IDLE: Re-enable color configuration for code context (GH-7199)
https://github.com/python/cpyt
Change by Terry J. Reedy :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
title: IDLE: Configurable color on code context -> IDLE: Re-enable color
configuration for code context
type: enhancement
Change by Terry J. Reedy :
--
stage: -> needs patch
versions: +Python 3.7, Python 3.8 -Python 3.5
___
Python tracker
<https://bugs.python.org/issu
Terry J. Reedy added the comment:
After writing the above, I realized that line scroll only touches editor.py,
not codecontext.py. On the other hand, the next item, 8. jump to context line
did have to wait. Replacing Label with Text will require explicit height
setting with each context
New submission from Terry J. Reedy :
Item 8. on #33610. Clicking on a line in the context box jumps the editor to
that line, displayed as the first line. If we do item 11, replace Label with
Text, this has to start after #33763 is merged.
We could determine the context line clicked from
Change by Terry J. Reedy :
--
dependencies: +IDLE: Use text widget for code context instead of label widget
nosy: +cheryl.sabella
___
Python tracker
<https://bugs.python.org/issue33
Terry J. Reedy added the comment:
This makes detecting the context line a user clicks on slightly easier. See
new issue #33768 for discussion. I am not sure now I would do this just for
this reason, without trying with the label first.
Using a text also allows us to experiment with
Terry J. Reedy added the comment:
New changeset d49dbd9acc6db544ca6cb2445fe17eb0c3be4bba by Terry Jan Reedy
(Cheryl Sabella) in branch 'master':
bpo-33664: Scroll IDLE editor text by lines (GH-7351)
https://github.com/python/cpython/commit/d49dbd9acc6db544ca6cb2445fe17e
Terry J. Reedy added the comment:
New changeset b609e687a076d77bdd687f5e4def85e29a044bfc by Terry Jan Reedy
(Cheryl Sabella) in branch 'master':
bpo-33763: IDLE: Replace label widget with text widget in code context (GH-7367)
https://github.com/python/cpyt
Terry J. Reedy added the comment:
I checked that with the patch, the wwindow top stayed synchronized with the
text after multiple movement operations of different types.
The minimum requirement for correct operation of the scrollbar slider is that
moving the slider to the top and bottom of
Terry J. Reedy added the comment:
I believe the colorizer would need some new code, maybe in a subclass, but it
should be possible. I would not know if I liked it until we tried it ;-).
--
resolution: -> fixed
stage: patch review -> resolved
status: open -&g
Terry J. Reedy added the comment:
Great. I might try some timing experiments later. I have seen both of the
first 2 options above in use.
--
___
Python tracker
<https://bugs.python.org/issue33
Terry J. Reedy added the comment:
This is a duplicate of the remaining part of #13153. The error message
displayed in a console used to start IDLE is the same, ending with
"UnicodeDecodeError: 'utf-8' codec can't decode byte 0xed in position 0:
invalid continuation by
Change by Terry J. Reedy :
--
Removed message: https://bugs.python.org/msg318863
___
Python tracker
<https://bugs.python.org/issue13153>
___
___
Python-bug
Change by Terry J. Reedy :
--
nosy: -JBernardo, Ramchandra Apte, Rosuav, THRlWiTi, William.Schwartz,
asvetlov, loewis, python-dev, roger.serwy
___
Python tracker
<https://bugs.python.org/issue13
Change by Terry J. Reedy :
--
pull_requests: -6157
___
Python tracker
<https://bugs.python.org/issue33274>
___
___
Python-bugs-list mailing list
Unsubscribe:
Terry J. Reedy added the comment:
I unlinked bad PR 6462 (against 3.6). PR 7465 is properly formed and should be
reviewed.
--
___
Python tracker
<https://bugs.python.org/issue33
Terry J. Reedy added the comment:
2.7 does not get enhancements. When a core dev removes a version, don't re-add
without discussion.
--
versions: -Python 2.7
___
Python tracker
<https://bugs.python.org/is
Terry J. Reedy added the comment:
Ari, I know that you viewed this as a behavior (bugfix) issue from the start
(message 1). I changed to enhancement (feature request) in message 2 because,
as you acknowledged in message 3, it is an edge case. The problem is
classification can be fuzzy
Terry J. Reedy added the comment:
Let me clarify part of what I wrote. If we make this change in 2.7.16, then
any code that uses the return value will not run on anything previous. This
may not matter for you, but would be bad for public code. This cost is present
in all bug fixes, but
Terry J. Reedy added the comment:
Summary for users: Previously, Code Context displayed a fixed number of lines.
Often, many were blank. Hence the default *numlines* to display was small, 3,
to avoid having too many blank lines. However, 3 is often too small to display
the complete
Terry J. Reedy added the comment:
Summary for users: Previously, there was one code context foreground and
background font color setting, default or custom, on the extensions tab, that
applied to all themes. Now, each theme has it own setting, as with other color
settings. For builting
Change by Terry J. Reedy :
--
title: IDLE: Re-enable color configuration for code context -> IDLE: Enable
theme-specific color configuration for code context
___
Python tracker
<https://bugs.python.org/issu
Change by Terry J. Reedy :
--
title: IDLE: Use variable number of lines in CodeContext -> IDLE: Display up to
maxlines non-blank lines for Code Context
___
Python tracker
<https://bugs.python.org/issu
Change by Terry J. Reedy :
--
pull_requests: +7125
___
Python tracker
<https://bugs.python.org/issue33642>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Terry J. Reedy :
--
pull_requests: +7126
___
Python tracker
<https://bugs.python.org/issue33679>
___
___
Python-bugs-list mailing list
Unsubscribe:
Terry J. Reedy added the comment:
New changeset bed523ba03c4525c9c79a6df700284b6c43024b3 by Terry Jan Reedy in
branch 'master':
bpo-33642 and bpo-33679: Revise and condense blurbs (GH-7500)
https://github.com/python/cpython/commit/bed523ba03c4525c9c79a6df700284
Terry J. Reedy added the comment:
New changeset bed523ba03c4525c9c79a6df700284b6c43024b3 by Terry Jan Reedy in
branch 'master':
bpo-33642 and bpo-33679: Revise and condense blurbs (GH-7500)
https://github.com/python/cpython/commit/bed523ba03c4525c9c79a6df700284
Terry J. Reedy added the comment:
New changeset 041272b657867f5bec925b19aabf23944125d49b by Terry Jan Reedy
(Cheryl Sabella) in branch 'master':
bpo-33768: IDLE: Clicking on code context line moves it to top of editor
(GH-7411)
https://github.com/python/cpyt
Change by Terry J. Reedy :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Terry J. Reedy :
--
pull_requests: +7145
___
Python tracker
<https://bugs.python.org/issue30928>
___
___
Python-bugs-list mailing list
Unsubscribe:
Terry J. Reedy added the comment:
The 3.6.6rc is next Monday. 3.6.8 will be the last 3.6 maintenance release.
--
nosy: +terry.reedy
___
Python tracker
<https://bugs.python.org/issue24
Terry J. Reedy added the comment:
Martin, the PR has one 'approved' review, not from you. You appear to be
requesting changes here, but it is not clear to me which are 'maybe' and which
are 'must'.
It is not obvious to me whether this should be treated as
Terry J. Reedy added the comment:
Martin, I added you because this Tempfile issue is related to #26175.
--
nosy: +martin.panter, terry.reedy
___
Python tracker
<https://bugs.python.org/issue33
Terry J. Reedy added the comment:
A few years ago, there was a particular case in which compile failed without a
trailing newline. We fixed it so that it would work anyway.
Unless we are willing for a conforming Python interpreter to fail
>>> exec('print("hello"
Terry J. Reedy added the comment:
Is the new message a clarification, in which case we would typically not
backport, or a correction, which we usually would?
--
nosy: +terry.reedy
___
Python tracker
<https://bugs.python.org/issue33
Terry J. Reedy added the comment:
This is a duplicate in that the 3 => 5 change was incorporated into PR 7419 for
#28240.
--
nosy: +terry.reedy
resolution: -> duplicate
stage: needs patch -> resolved
status: open -> closed
superseder: -> Enhance the timeit module:
Change by Terry J. Reedy :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Terry J. Reedy added the comment:
Westly, #31500 changed font scaling for IDLE on high DPI screens. It also
removed some hard-coding of frame sizes in terms of pixels. Does anything look
better for you with current IDLE (3.6.5, 3.7.0
Terry J. Reedy added the comment:
Elisha wrote and I quoted: '[the call] sets DPI awareness'. After reading
https://msdn.microsoft.com/en-us/library/windows/desktop/dn280512(v=vs.85).aspx
PROCESS_DPI_AWARENESS, what the call sets is the system's idea of what DPI
awareness
Terry J. Reedy added the comment:
Do any of the Windows experts know what "Override high DPI scaling behavior" on
properties does? Does it call SetProcessDpiAwareness? It seems so. The answer
would have implications for #33656 (which is about fixing this issue in IDLE,
Terry J. Reedy added the comment:
Elisha Paine, who prompted #33656 by posting the SetProcessDpiAwareness(1) fix
to the idledev list, wrote to me that pyglet and probably pygame have similar
issues.
I wonder if Python should be installed with the 'application' setting, or if
DPI
Terry J. Reedy added the comment:
As reported on #26698, which was originally opened as an IDLE issue, but which
I turned into a tkinter issue, one can also sharpen text buy opening
python(w).exe properties, Compatibility tab, clicking [Change high DPI
settings], and then checking
Terry J. Reedy added the comment:
Eric, thanks for the additional reference. From Window's viewpoint, Python,
not IDLE, is the application. IDLE is just input data. I should not touch
Python's manifest, and changing it would have to consider other possible gui
module input d
7301 - 7400 of 13135 matches
Mail list logo
