Patches item #1460496, was opened at 2006-03-28 23:30 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1460496&group_id=5470
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Core (C code) Group: Python 2.5 Status: Open Resolution: None Priority: 5 Submitted By: Wesley J. Chun (wesc) Assigned to: Nobody/Anonymous (nobody) Summary: give round() kwargs (mostly for PFAs) Initial Comment: 1. WHAT this is a short non-critical patch to give round() keyword arguments for Partial Function Application (PEP309) purposes. 2. WHY arguments that appear on the RHS of function declarations are hard to call when using functional.partial because the "variable" argument is to the left (since PFAs pretty much favor only variable arguments to the right). if fixed arguments are to the right (are "curried"), then we need to support keyword args for them as much as possible to encourage their use. 3. MOTIVATION similar to the example in the Python Lib Ref doc page for the functional module and its use with int()... http://docs.python.org/dev/lib/module-functional.html >>> basetwo = partial(int, base=2) >>> basetwo.__doc__ = 'Convert base 2 string to an int.' >>> basetwo('10010') 18 ...we should also allow round() to be used in the same way... >>> from functional import partial >>> >>> roundTwo = partial(round, ndigits=2) >>> print '%.2f' % roundTwo(1.5555555) 1.56 >>> print '%.2f' % roundTwo(1.5545555) 1.55 >>> print '%.2f' % roundTwo(1.5445555) 1.54 without this patch, users current get this: >>> from functional import partial >>> roundTwo = partial(round, ndigits=2) >>> '%.2f' % roundTwo(1.5555555) Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: round() takes no keyword arguments 4. THE FIX the number variable in bltinmodule.c is renamed from 'x' to 'number', and the keyword is named 'ndigits' after the doc string: >>> print round.__doc__ round(number[, ndigits]) -> floating point number this patch consists of a single file, Python/bltinmodule.c. the original CVS version from Mar 28, the patched version, output from running test/test_builtin.py, and both the C and U diff files are included in the attached ZIP file. 5. TESTING as mentioned above, we ran the test/test_builtin.py script and there are no changes in the execution of test_round: : test_repr (test.test_builtin.BuiltinTest) ... ok test_round (test.test_builtin.BuiltinTest) ... ok test_setattr (test.test_builtin.BuiltinTest) ... ok : the full output file is in the ZIP file. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1460496&group_id=5470 _______________________________________________ Patches mailing list [email protected] http://mail.python.org/mailman/listinfo/patches
