classification
Title: format error messages should provide context information
Type: behavior Stage: test needed
Components: Library (Lib) Versions:
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: eric.smith, ezio.melotti, r.david.murray
Priority: normal Keywords:

Created on 2014-02-05 23:23 by r.david.murray, last changed 2014-02-06 01:08 by eric.smith.

Messages (3)
msg210351 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2014-02-05 23:23
Consider the following:

   '{run_time:%H:%M:%S}, ,COM,DA{id},"{title:.43}",{id},{length:%M:%S}'.format(**mydict)

The error message I got was:

   Invalid format specifier

The problem turned out to be that the value of the 'length' key was an integer instead of a datetime.time(), but it sure wasn't easy to figure out which bit of the format string or which variable was the problem.

It would be nice for the format error message to include the pattern that it is parsing when it hits the error.  The type of the value being substituted would also be nice.  Perhaps something like:

   The format specifier in {length:%HH:%MM} is not valid for type int()
msg210354 - (view) Author: Eric V. Smith (eric.smith) * (Python committer) Date: 2014-02-06 00:32
That would be a great improvement. It's in Python/formatter_unicode.c, line 245, in parse_internal_render_format_spec().

That code knows about the format spec, but not the type being formatted. That would be easy enough to pass in.

This fix would only work for the built-in types: int, float, and str, I think. Maybe complex. But that's probably good enough.
msg210359 - (view) Author: Eric V. Smith (eric.smith) * (Python committer) Date: 2014-02-06 01:08
int, float, str, and complex are the types formatted by that code.

Notice that Decimal already has a better message:

>>> format(Decimal(42), 'tx')
Traceback (most recent call last):
  ...
ValueError: Invalid format specifier: tx

>>> format(42, 'tx')
Traceback (most recent call last):
  ...
ValueError: Invalid conversion specification

But, look at this:
>>> format(3, '--')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: Unknown format code '-' for object of type 'int'

This is generated in unknown_presentation_type, also in formatter_unicode.c. It almost does what you want, but just handles the presentation type, not the whole format specifier.

Error handling could be cleaned up in that module. I'd say that the string should be:
"<specific error> with format specifier <specifier> for object of type '<type>'"

<specific error> might be "Unknown presentation type '-'", or "Cannot specify ','".

I think that would require some major surgery to the code, but would be worth it.

Note that in your original example, you want the error to contain "{length:%HH:%MM}". By the time the error is detected, the only thing the code knows is the format specifier "%HH:%MM". It doesn't know the "length" part. The error is basically in int.__format__. By the time that gets called, the format specifier has already been extracted and the argument selection (by indexing, by name, including attribute access) has already taken place.
History
Date User Action Args
2014-02-06 01:08:15eric.smithsetmessages: + msg210359
2014-02-06 00:32:46eric.smithsetmessages: + msg210354
2014-02-05 23:36:05ezio.melottisetnosy: + ezio.melotti

type: behavior
stage: test needed
2014-02-05 23:23:15r.david.murraycreate