Skip to content

Corrections to format precision description.#31291

Merged
miss-islington merged 2 commits into
python:mainfrom
belm0:format_precision_docs
Feb 14, 2022
Merged

Corrections to format precision description.#31291
miss-islington merged 2 commits into
python:mainfrom
belm0:format_precision_docs

Conversation

@belm0
Copy link
Copy Markdown
Contributor

@belm0 belm0 commented Feb 12, 2022 •
edited by miss-islington
Loading

  • precision field is a decimal integer
  • clarify that stated limitations are on presentation type
    rather than input value type. Especially misleading is
    "precision is not allowed for integer values", since integer
    value input to a format like .1f is fine.
  • regarding max field size, replace "non-number" with "string",
    which is the only non-numeric presentation type

Automerge-Triggered-By: GH:ericvsmith

@belm0
Corrections to format precision description.
2f0c075 
  * `precision` field is an integer, not decimal
  * clarify that stated limitations are on presentation type
    rather than input value type.  Especially misleading is
    "precision is not allowed for integer values", since integer
    value input to a format like `.1f` is fine.
  * regarding max field size, replace "non-number" with "string",
    which is the only non-numeric presentation type
@bedevere-bot bedevere-bot added the docs Documentation in the Doc dir label Feb 12, 2022
@belm0
Copy link
Copy Markdown
Contributor Author

belm0 commented Feb 13, 2022

@ericvsmith would you mind reviewing this doc correction?

ericvsmith
ericvsmith reviewed Feb 13, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@ericvsmith ericvsmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change looks really good. Thanks. @mdickinson, any comments?

As long as we're changing this section, should 'e' and 'E' be mentioned?

@mdickinson
Copy link
Copy Markdown
Member

mdickinson commented Feb 13, 2022

Changes look good to me. How about "decimal integer" in place of "integer"? (Strictly speaking, the precision isn't an integer - it's a decimal string representation of an integer.)

FWIW, "decimal integer" is the language that the C specification uses here. From C99:

The precision takes the form of a period (.) followed either by an asterisk * (described later) or by an optional decimal integer [...]

@mdickinson
Copy link
Copy Markdown
Member

mdickinson commented Feb 13, 2022

As long as we're changing this section, should 'e' and 'E' be mentioned?

Probably, yes, though it would be fine to change it in a follow-up PR. Technically, the same description as for 'f' and 'F' applies, I think. (Though this feels a bit misleading, given that the precision has quite a different meaning.)

@ericvsmith
Copy link
Copy Markdown
Member

ericvsmith commented Feb 13, 2022

As long as we're changing this section, should 'e' and 'E' be mentioned?

Probably, yes, though it would be fine to change it in a follow-up PR.

I'm okay with it being a separate PR.

So other than "decimal integer", this is ready to go. Note that for width it already says: "width is a decimal integer".

ericvsmith
ericvsmith approved these changes Feb 14, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@ericvsmith ericvsmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@miss-islington miss-islington merged commit 1d6ce67 into python:main Feb 14, 2022
@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Feb 14, 2022

Thanks @belm0 for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 14, 2022
@belm0 @miss-islington
Corrections to format precision description. (pythonGH-31291)
3ca8848 
* `precision` field is a decimal integer
  * clarify that stated limitations are on presentation type
    rather than input value type.  Especially misleading is
    "precision is not allowed for integer values", since integer
    value input to a format like `.1f` is fine.
  * regarding max field size, replace "non-number" with "string",
    which is the only non-numeric presentation type

Automerge-Triggered-By: GH:ericvsmith
(cherry picked from commit 1d6ce67)

Co-authored-by: John Belmonte <john@neggie.net>
@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Feb 14, 2022

Sorry, @belm0, I could not cleanly backport this to 3.9 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 1d6ce67c29aa2166ef326952cb605b908fb4f987 3.9

@miss-islington miss-islington self-assigned this Feb 14, 2022
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Feb 14, 2022

GH-31320 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Feb 14, 2022
@belm0 belm0 deleted the format_precision_docs branch February 14, 2022 00:07
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Feb 14, 2022

GH-31321 is a backport of this pull request to the 3.9 branch.

belm0 added a commit to belm0/cpython that referenced this pull request Feb 14, 2022
@belm0
[3.9] Corrections to format precision description. (pythonGH-31291)
b6aeac6 
* `precision` field is a decimal integer
  * clarify that stated limitations are on presentation type
    rather than input value type.  Especially misleading is
    "precision is not allowed for integer values", since integer
    value input to a format like `.1f` is fine.
  * regarding max field size, replace "non-number" with "string",
    which is the only non-numeric presentation type

Automerge-Triggered-By: GH:ericvsmith.
(cherry picked from commit 1d6ce67)

Co-authored-by: John Belmonte <john@neggie.net>
miss-islington added a commit that referenced this pull request Feb 14, 2022
@miss-islington @belm0
Corrections to format precision description. (GH-31291)
6a7dd3f 
* `precision` field is a decimal integer
  * clarify that stated limitations are on presentation type
    rather than input value type.  Especially misleading is
    "precision is not allowed for integer values", since integer
    value input to a format like `.1f` is fine.
  * regarding max field size, replace "non-number" with "string",
    which is the only non-numeric presentation type

Automerge-Triggered-By: GH:ericvsmith
(cherry picked from commit 1d6ce67)

Co-authored-by: John Belmonte <john@neggie.net>
ericvsmith pushed a commit that referenced this pull request Feb 14, 2022
@belm0
[3.9] Corrections to format precision description. (GH-31291) (GH-31321)
0b1417d 
* `precision` field is a decimal integer
  * clarify that stated limitations are on presentation type
    rather than input value type.  Especially misleading is
    "precision is not allowed for integer values", since integer
    value input to a format like `.1f` is fine.
  * regarding max field size, replace "non-number" with "string",
    which is the only non-numeric presentation type

Automerge-Triggered-By: GH:ericvsmith.
(cherry picked from commit 1d6ce67)

Co-authored-by: John Belmonte <john@neggie.net>
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
@belm0 @hello-adam
[3.9] Corrections to format precision description. (pythonGH-31291) (p…
dbb4b48 
…ythonGH-31321)

* `precision` field is a decimal integer
  * clarify that stated limitations are on presentation type
    rather than input value type.  Especially misleading is
    "precision is not allowed for integer values", since integer
    value input to a format like `.1f` is fine.
  * regarding max field size, replace "non-number" with "string",
    which is the only non-numeric presentation type

Automerge-Triggered-By: GH:ericvsmith.
(cherry picked from commit 1d6ce67)

Co-authored-by: John Belmonte <john@neggie.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ericvsmith ericvsmith ericvsmith approved these changes

Assignees

@miss-islington miss-islington

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@belm0 @mdickinson @ericvsmith @miss-islington @bedevere-bot @the-knights-who-say-ni