Yoni Lavi added the comment:
Thanks for the good feedback everyone and apologies for the unresponsiveness
over the past day.
I understand that my use cases may not reflect wider usage patterns and am not
looking to argue against the closing. But anyway, for future reference, I'll
add two
Change by Yoni Lavi :
--
keywords: +patch
pull_requests: +17124
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/17657
___
Python tracker
<https://bugs.python.org/issu
New submission from Yoni Lavi :
I would like to put forward an argument in favour of a `default` parameter in
the statistics.mean function and the related function.
What motivated me to open this is that my code would more often than not
include a check (or try-except) whenever I calculate
Yoni Lavi added the comment:
Created issue 27304 and uploaded a patch.
--
___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue22558>
___
__
New submission from Yoni Lavi:
This is a follow to #22558.
As per Terry's suggestion in #22558, for packages with multiple modules that
are all documented on the same page and that currently have a "Source Code"
link to an __init__ file (e.g. importlib), we should also add mor
Yoni Lavi added the comment:
Thank you Terry, Nathan and David for your work on this.
Sorry about the whitespace issue, I indeed forgot to run patchcheck when I
prepared the 3rd patch.
Tery, would you like me to prepare a patch for the follow-on you suggested? If
so, given that this one
Yoni Lavi added the comment:
Thank you very much for the detailed review, Nathan. Attached is a new patch
that resolves the merge issues, and some of the other issues you mentioned.
Concerning the index reference in marshal.rst , ah, yes, I missed that it only
related to a single paragraph
Yoni Lavi added the comment:
Ah, thanks for the explanation. I think in this case we're ok too.
I linked io.rst to Lib/io.py, which relies on the C version. And in the case of
decimal.rst, it was already linked to Lib/decimal.py which very explicitly
attempts to load the C version and falls
Yoni Lavi added the comment:
Thank you for looking at this.
David, concerning modules that have multiple sources, are you referring to
modules such as `stat` and `heapq`? In these, it's the Python source that tries
to import the C module if it's available, so I'd consider it ok to leave just
Yoni Lavi added the comment:
Ping. Please advise on next step to get this submitted.
--
___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/i
Yoni Lavi added the comment:
Thank you for the review, Berker.
I removed source code links from all the modules you specifically mentioned,
and all links to C code, including stat.rst which already had such a link
before.
But I have to say that I don't quite understand the rationale here
Yoni Lavi added the comment:
Friendly ping
--
___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue22558>
___
___
Python-bugs-list
Yoni Lavi added the comment:
I went over all the library rst files and added a link to the source for each
of the modules.
Also, I standardized the structure of the headers in all of the files to be in
the following order: """
.. module
.. moduleauthor
.. sectionauthor
Changes by Yoni Lavi <y...@lavi.fm>:
--
nosy: +Yoni Lavi
___
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue22558>
___
___
Python
14 matches
Mail list logo
