Stéphane Blondon added the comment:
I can reproduce it on python 3.8.
I can't reproduce it on python 3.9 or 3.10.
--
nosy: +sblondon
___
Python tracker
<https://bugs.python.org/is
Stéphane Blondon added the comment:
I can reproduce the error on Debian/testing with:
- Python 3.9.9
- Python 3.10.0
--
nosy: +sblondon
status: pending -> open
___
Python tracker
<https://bugs.python.org/issu
Change by Stéphane Blondon :
--
keywords: +patch
pull_requests: +23624
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/24864
___
Python tracker
<https://bugs.python.org/issu
Stéphane Blondon added the comment:
Thank you Felipe for the news! :)
I have committed a PR about this issue.
Two remarks:
- I changed the proposed implementation from 'format(integer, '_d')' to
'{:_d}.format(integer)' because the first way raised an exception. (
Stéphane Blondon added the comment:
I changed the default to be backward compatible (so underscore_numbers=False).
I think it would be better with underscore_numbers enabled by default but I
understand the need for stability. Perhaps such break could be done in the
future (in version 3.12 or
Change by Stéphane Blondon :
--
pull_requests: +24445
stage: backport needed -> patch review
pull_request: https://github.com/python/cpython/pull/25755
___
Python tracker
<https://bugs.python.org/issu
Stéphane Blondon added the comment:
It seems ezio-melotti hesitates to continue the modifications in the PR
https://github.com/python/cpython/pull/21552#pullrequestreview-458400056
IMO, the PR improves enough the documentation, so it would be nice to merge it,
even without the other changes
Stéphane Blondon added the comment:
I think an asdict() method is not necessary. Instanciating a dict from
os.environ is enough:
>>> import os
>>> os.environ
environ({'SHELL': '/bin/bash', ...})
>>> dict(os.environ)
{'SHELL': '/
New submission from Stéphane Blondon :
Currently, logging.root.manager.loggerDict is usable to do a homemade traversal
of the loggers tree. However, it's not a public interface. Adding a
'logger.getChildren()' method would help to implement the traversal. The method
would
Stéphane Blondon added the comment:
Python 3.10 has now been released with the underscore_numbers parameter.
I wonder which release could enable the parameter by default (so it would break
the previous behavior):
- the next release (3.11) is probably too short.
- the safest strategy is to
Stéphane Blondon added the comment:
Ok, I will not send a PR to change the current behavior until python4 (in case
it exists one day).
--
___
Python tracker
<https://bugs.python.org/issue42
Change by Stéphane Blondon :
--
keywords: +patch
nosy: +sblondon
nosy_count: 5.0 -> 6.0
pull_requests: +22720
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/23855
___
Python tracker
<https://bugs.python.org/i
Change by Stéphane Blondon :
--
keywords: +patch
nosy: +sblondon
nosy_count: 2.0 -> 3.0
pull_requests: +22734
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/23872
___
Python tracker
<https://bugs.python.org/i
Stéphane Blondon added the comment:
Orian: your patch formats the error message but the original suggested
solution is to store the missing key in a new attribute.
I don't know if you go in the good direction.
Adding an attribute is also suggested by issue #614557.
--
Stéphane Blondon added the comment:
Merged by Łukasz Langa in september 2020.
(Thanks Łukasz)
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Stéphane Blondon added the comment:
RangeHTTPServer seems to support python3: there is a try-except clause to
manage the different import of SimpleHTTPServer:
https://github.com/danvk/RangeHTTPServer/blob/master/RangeHTTPServer/__main__.py#L13
The code is under Apache 2.0 licence. I don
Stéphane Blondon added the comment:
I'm interested by such feature.
I see examples of versions of the message provided by KeyError:
- sometimes with a message (for example `PyErr_SetString(PyExc_KeyError, "name
too long");` at [1])
- sometimes with the missing k
Stéphane Blondon added the comment:
I add the same idea but later than you, so I'm interested by such feature.
Felipe: do you want to add a pull request to this issue (with Serhiy Storchaka
implementation because it's the simplest one)?
If not, I plan to write it.
I will write
New submission from Stéphane Blondon :
The documentation is not explicit about the behaviour if several files are read
by the same ConfigParser: the data are not reset between two read().
I suggest to add such information in the documentation. There is a draft:
=== start ===
When a
Change by Stéphane Blondon :
--
keywords: +patch
pull_requests: +14908
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/15177
___
Python tracker
<https://bugs.python.org/issu
Stéphane Blondon added the comment:
Thank you for the idea. :)
I committed something else, based on the previous example of the 'Quick Start'
section.
What do you think about it?
--
___
Python tracker
<https://bugs.python.o
Stéphane Blondon added the comment:
I don't know about something blocking the merge. I sent a message to Barry
on Github the 3th january but perhaps it was a wrong timing (too soon after
new year).
--
___
Python tracker
<https://bugs.py
Stéphane Blondon added the comment:
The availability of context manager does not make it mandatory if it does
not fit some needs.
> In the last example on https://docs.python.org/3/
> library/mailbox.html#examples inbox is locked and unlocked multiple
> times. The with statement co
New submission from Stéphane Blondon :
In several cases, tests use ```self.assertTrue(a in b)```. Using
```self.assertIn(a, b)``` seems to be better.
For examples:
./Lib/test/test_inspect.py:
self.assertTrue('(po, pk' in repr(sig))
./Lib/test/test_configparser.py:
self.
New submission from Stéphane Blondon :
The html produced by cgitb.html() does not display because the square
brackets are interpreted as a html tag (see the picture in attachement).
This bug occurs if the code is called directly in the module, not inside a
function or a class
Change by Stéphane Blondon :
--
type: -> behavior
___
Python tracker
<https://bugs.python.org/issue33256>
___
___
Python-bugs-list mailing list
Unsubscrib
Change by Stéphane Blondon :
--
keywords: +patch
pull_requests: +6138
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue33256>
___
___
Py
New submission from Stéphane Blondon :
The cgitb module displays a traceback in text or html.
When a module is called, there are no parameters (displayed as '()'). I
think they are unnecessary and the parentheses should be removed.
### example for the text version ###
$ pytho
Change by Stéphane Blondon :
--
keywords: +patch
pull_requests: +6215
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue33311>
___
___
Py
Change by Stéphane Blondon :
--
pull_requests: +6216
___
Python tracker
<https://bugs.python.org/issue3356>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Stéphane Blondon :
--
pull_requests: +6371
___
Python tracker
<https://bugs.python.org/issue33311>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Stéphane Blondon :
cgitb.html() returns HTML code viewable in a browser.
It would be nice to improve the rendering to have something more readable
(increase space between each call, increase contrast for the source code) and
more modern.
You can see the difference in the
Stéphane Blondon added the comment:
current.png is a screenshot of the html shown in a browser with 3.8 branch
next.png is a screenshot of the modified cgitb.html() function.
--
Added file: https://bugs.python.org/file47589/current.png
___
Python
Change by Stéphane Blondon :
--
keywords: +patch
pull_requests: +6529
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue33507>
___
___
Py
Stéphane Blondon added the comment:
> Instead of hardcoding colors, would it be better to use CSS instead.
I agree with you. The current source code contains hardcoded colors. I
could add an optional parameter to change the css for the cgitb.html()
function:
def html(einfo, context=5,
Stéphane Blondon added the comment:
I added the optional parameter to cgitb.html() function and removed the
hardcoded css from the html code.
I think there are still several improvements to do:
- add documentation for the new parameter of cgitb.html()
- add the same parameter to cgitb.enable
36 matches
Mail list logo