New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
format error messages should provide context information #64723
Comments
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() |
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. |
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> 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. |
Reproduced on 3.11: >>> dd = {'length': 12, 'id': 4, 'title': "t", 'run_time': datetime.time()}
>>> '{run_time:%H:%M:%S}, ,COM,DA{id},"{title:.43}",{id},{length:%M:%S}'.format(**dd)
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
ValueError: Invalid format specifier |
I've decided to take an easy path: #28310
This worked for me as the starting point. It covered: int, float, complex, and str types.
I guess it has changed since the 2014. I would love to hear any feedback on my proposal and improve it to the level when it can be merged :) |
Thanks for the improvement, @sobolevn! |
.format()
#28310Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: