-
-
Notifications
You must be signed in to change notification settings - Fork 30.8k
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
The Great Argument Clinic Conversion Derby Meta-Issue #64386
Comments
This is a meta-issue tracking all the Argument Clinic Derby sub-issues. |
The issue tracker isn't letting me add all the dependencies to the dependency list. For the record, here are the numbers: bpo-20133 (so far) |
Here's how the files were parceled out into the various bundles: bpo-20170
bpo-20171
bpo-20172
bpo-20173
bpo-20174
bpo-20175
bpo-20159
bpo-20177
bpo-20178
bpo-20179
bpo-20180
bpo-20181
bpo-20182
bpo-20183
bpo-20152
bpo-20184
bpo-20185
bpo-20186
|
I want to warn against the converting of base objects (whose sources lie in Objects/). Their performance is critical and the converting can worsen it. Often they even don't use PyArg_ParseTuple due to performance reasons or because argument parsing is too special for PyArg_ParseTuple. I would suggest to postpone it until Argument Clinic will generate optimized parsing code. The same is true for some modules. The performance of struct and operator is important. In any case these modules almost not use PyArg_ParseTuple and can be omitted. |
Can you give me an example of performance degradation due to Argument Clinic? What percentage slowdown is common? |
I have no any numbers. |
Until you have some numbers, your assertion has no weight and should be ignored. (As Carl Sagan once wrote: "That which can be asserted without evidence can be dismissed without evidence." And as Donald Knuth wrote: "Premature optimization is the root of all evil.") |
I've been working on converting unicode, bytes and bytearray, but their files are separated into Derby groups #10, #11 and #12. Could we create a new issue and move the relevant files to it? The files I'd like to move: From Derby #10 (bpo-20179): From Derby #11 (bpo-20180): From Derby #12 (bpo-20181): |
Never mind my previous comment; I realize it would create to much of a mess. Sorry for the noise. |
@larryhastings Is this issue still relevant? All the AC PRs listed in the header are either merged or closed. A |
Individual issue can be created if required as this is completed. |
Note: 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: