homepage

This issue tracker has been migrated to GitHub , and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: tracemalloc: rename "max_size" to "peak_size" in get_traced_memory() result
Type: Stage:
Components: Library (Lib) Versions: Python 3.4
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: neologix, python-dev, vstinner
Priority: normal Keywords: patch

Created on 2013年11月26日 12:55 by vstinner, last changed 2022年04月11日 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
peak_size.patch vstinner, 2013年11月26日 12:55 review
Messages (5)
msg204470 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2013年11月26日 12:55
Jim.J.Jewett asked me to replaced "max_size" with "peak_size". It looks like Linux also uses the term "peak" for /proc/pid/status (or sometimes "High Water Mark").
See attached patch.
msg204473 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2013年11月26日 13:00
@neologix: I would like your opinion first, because you accepted the PEP.
msg204476 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2013年11月26日 13:37
> STINNER Victor added the comment:
>
> @neologix: I would like your opinion first, because you accepted the PEP.
Well, I'm not a native speaker, but "peak" does sound better than
"max" (I'd say the later refers to an externally-enforced limit,
contrarily to the former).
So the change looks fine, although:
"""
`(size: int, peak_size: int)``
"""
Would maybe be better with:
"""
`(current: int, peak: int)``
"""
no?
msg204610 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2013年11月27日 20:40
New changeset 553144bd7bf1 by Victor Stinner in branch 'default':
Close #19798: replace "maximum" term with "peak" in get_traced_memory()
http://hg.python.org/cpython/rev/553144bd7bf1 
msg204611 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2013年11月27日 20:41
> no?
yes, I also prefer "current" over "size".
History
Date User Action Args
2022年04月11日 14:57:54adminsetgithub: 63997
2013年11月27日 20:41:04vstinnersetmessages: + msg204611
stage: resolved ->
2013年11月27日 20:40:11python-devsetstatus: open -> closed

nosy: + python-dev
messages: + msg204610

resolution: fixed
stage: resolved
2013年11月26日 13:37:34neologixsetmessages: + msg204476
2013年11月26日 13:00:04vstinnersetmessages: + msg204473
2013年11月26日 12:55:08vstinnersetcomponents: + Library (Lib)
versions: + Python 3.4
2013年11月26日 12:55:01vstinnercreate

AltStyle によって変換されたページ (->オリジナル) /