GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||
GeezerVison is showing github.com — gh-140382: Add support for `__getattr__` method to Argument Clinic by StanFromIreland · Pull Request #140376 · python/cpython Conversation
|
|
Could you create a new issue for this, and tag people interested in AC? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I can create a new issue, but all CODEOWNERs have already been notified. Do you have anyone else in mind? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
__getattr__ method to Argument Clinic__getattr__ method to Argument Clinic
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't it make more sense to just support arbitrary dunder methods? __getattr__ isn't very common in C code, among other things.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
I'm also not convinced this is worth it. A |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
On the other hand, I would be very supportive of allowing arbitrary dunders in AC, though I suspect that would be a lot of extra work (because we would need to map dunders to |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
No, it’s just the general principle that «should we do?» discussions happen in issues (and when needed on the forums), then the PR can host «how to do?» discussions. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't it make more sense to just support arbitrary dunder methods?
I think it's ok to add support of specific dunders on-demand. It's not too helpful in general, because sometimes we can do too much with slots, even override the docstring.
But please think about more generic case. For example, for most dunders we have very specific function signatures. It will be redundant and error-prone to specify this stuff by hand.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
I would prefer to only add support when it will really be needed, so we can test on actual code rather than unit tests. In short, I'm supportive of only adding support for |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I don't think that will work well in practice. Judging by this PR, adding support for a new dunder method seems non-trivial. People will probably decide to just write things the old way instead of updating clinic, which we don't want. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Closing, there is little need and support for this. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.