[issue22672] float arguments in scientific notation not supported by argparse
Changes by Evgeny Kapun abacabadabac...@gmail.com: -- nosy: +abacabadabacaba ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
Changes by Terry J. Reedy tjre...@udel.edu: -- resolution: - duplicate stage: patch review - resolved superseder: - argparse does not accept options taking arguments beginning with dash (regression from optparse) ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
paul j3 added the comment: closed with reference to #9334 -- status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
Changes by Ned Deily n...@acm.org: -- nosy: -ned.deily ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
Terry J. Reedy added the comment: If this issue is a duplicate of #9334, as it appears to be, this should be closed and a note added to #9334 that there is a patch here also. -- nosy: +terry.reedy ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
Jacopo Nespolo added the comment: Ned Deily: I don't quite know how to use unittest, but I'll try to look into it. paul j3: There I proposed leaving '_negative_number_matcher' unchanged, but only use it to set '_has_negative_number_optionals'. I don't know argparse internals but, if I understand your argument, I think you still need a '_negative_number_matcher' capable of correctly match any negative number in any form, including scientific notation, complex j, etc.. If this is the case, then, the matcher could be simplified to something like '^-\d+|^-.\d+' (notice $ removed from current regex) which would only signal that something that starts like a negative number is present. Then one could use complex() or float() or whatever, to check that that's actually the case. I would still expect an exception to be raised if, say, I specify type=float and then a complex value is passed as an argument. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
paul j3 added the comment:
Patch tests are added to an existing 'Lib/test/test_argparse.py' file. I use
existing test cases as a pattern any new tests.
-
Your test file runs fine with the patch I proposed for Issue 9334.
-
The argparse code uses '_negative_number_matcher' for 2 purposes
1) to test whether an option_string looks like a negative number, and set the
'self._has_negative_number_optionals' attribute.
parser.add_argument('-2')
parser.add_argument('-1.234')
2) to test whether an argument string (an element of sys.argv) looks like one
of those option_strings, or is an argument (positional or argument to your
'-a').
The 'type' in for your '-a' argument is separate issue. That is used convert a
string to a number, float, complex or what ever, and raise an error it if can't
do so.
In your case './test.py -a -1e5' fails because '-1e5' fails the 2nd test, and
thus is not recognized as an argument to '-a'. Understanding the details of
this requires digging into the logic of the _parse_optional() method.
'./test.py -a-1e5' or './test.py -a=-1e5' work because the number is correctly
recognized as an argument.
For issue 9334 I looked at generalizing '_negative_number_matcher' as you did.
But unless you want to use something like:
parser.add_argument('-1.2e-34')
and set the 'self._has_negative_number_optionals' to '[True]', the matcher
doesn't need to be more general.
It's only the test in '_parse_optional()' that needs to be generalized to
handle scientific and complex notation. And for that I figured that wrapping
'complex()' in a 'try' block was just as general and reliable as a complicated
're' pattern. At least that was my claim in issue 9334, and I haven't gotten
feedback on that.
I'd suggest reading the 9334 discussion, and testing that patch. That patch
includes some tests for scientific and complex numbers.
That issue and patch also adds a 'args_default_to_positional' parameter. I
wonder if the two changes should be put in separate patches.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22672
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
Jacopo Nespolo added the comment: a better patch -- Added file: http://bugs.python.org/file37007/argparse.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
Jacopo Nespolo added the comment: test program that checks handling of a bunch of numbers. -- Added file: http://bugs.python.org/file37008/test_argparse.py ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
Ned Deily added the comment: Thanks for the updated patch and test program. Note, to be part of the Python test suite, the test should be in the form of a unittest test and fit into the existing argparse tests here: Lib/test/test_argparse.py. -- nosy: +ned.deily ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
Changes by Ned Deily n...@acm.org: -- stage: - patch review versions: +Python 3.5 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
paul j3 added the comment: This issue has already been raise in http://bugs.python.org/issue9334 argparse does not accept options taking arguments beginning with dash (regression from optparse) There I proposed leaving '_negative_number_matcher' unchanged, but only use it to set '_has_negative_number_optionals'. Testing whether argument strings are numbers or not is better done with 'complex()'. if not self._has_negative_number_optionals: try: complex(arg_string) return None except ValueError: pass That was my first patch, and I haven't looked at it much since then, other than testing that it is compatible with a number of other proposed patches. -- nosy: +paul.j3, r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
Changes by Jacopo Nespolo j.nesp...@gmail.com: -- nosy: +bethard ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22672 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue22672] float arguments in scientific notation not supported by argparse
New submission from Jacopo Nespolo:
Argparse fails to recognise negative numbers in scientific notation as valid
numbers.
Example:
Suppose the program test.py has this option.
par.add_argument('-a', type=float)
then './test.py -a -1e5' will fail, as well as -1.0e-4, -.5E+4 and variations
thereof.
Furthermore, at the current state, it seems that argparse does not recognize
-1. as a valid float either.
I tried to hack argparse.py myself, and I believe the patch attached should fix
this issue. The base version of argparse.py is the one from Python 3.4.2 as
found in Debian Sid.
The modified regular expression seemed to behave correctly in all test cases I
could come up with.
--
components: Library (Lib)
files: argparse.patch
keywords: patch
messages: 229689
nosy: jnespolo
priority: normal
severity: normal
status: open
title: float arguments in scientific notation not supported by argparse
type: enhancement
versions: Python 3.4
Added file: http://bugs.python.org/file36972/argparse.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22672
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
