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-104050: Annotate Argument Clinic parameter permutation helpers by erlend-aasland · Pull Request #106431 · python/cpython Conversation
|
|
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
:( I'm sorry to hear that. There's definitely a lot of concepts to grapple with, I'll fully concede. We have uncovered at least two real bugs along the way, though! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
No worries, I'm having fun (slowly learning)! And I think this whole project will end up increasing the readability and maintainability of clinic.py. I'm just frustrated when I can't figure out new stuff, nothing else 😄 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Also, once again, thanks for helping reviewing and explaining! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
So, here's what mypy complains about on my computer: Tools/clinic/clinic.py:536: error: Incompatible types in assignment (expression has type "list[Iterable[Parameter] | Parameter]", variable has type "list[Iterable[Parameter]]") [assignment] Tools/clinic/clinic.py:553: error: Argument 1 to "extend" of "list" has incompatible type "Iterable[Parameter]"; expected "Iterable[Iterable[Parameter]]" [arg-type] Tools/clinic/clinic.py:584: error: Argument 1 to "append" of "list" has incompatible type "tuple[Iterable[Parameter], ...]"; expected "Iterable[Parameter]" [arg-type] Tools/clinic/clinic.py:586: error: Argument "key" to "sort" of "list" has incompatible type "Callable[[Sized], int]"; expected "Callable[[Iterable[Parameter]], SupportsDunderLT[Any] | SupportsDunderGT[Any]]" [arg-type] Is mypy fooled by the |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
Humm, I wrote you a small essay above there, but I'm still working on getting mypy to actually pass with my suggestions... I'll get back to you :) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
Okay! Here's a diff that makes mypy happy, and I think it has the added virtue of being correct. How does it look to you? Details--- a/Tools/clinic/clinic.py +++ b/Tools/clinic/clinic.py @@ -28,7 +28,7 @@ import textwrap import traceback -from collections.abc import Callable +from collections.abc import Callable, Iterable, Iterator, Sequence from types import FunctionType, NoneType from typing import ( Any, @@ -516,7 +516,11 @@ class PythonLanguage(Language): checksum_line = "#/*[{dsl_name} end generated code: {arguments}]*/" -def permute_left_option_groups(l): +ParamGroupSequence = Sequence[Iterable["Parameter"]] +ParamGroupTuple = tuple["Parameter", ...] + + +def permute_left_option_groups(l: ParamGroupSequence) -> Iterator[ParamGroupTuple]: """ Given [(1,), (2,), (3,)], should yield: () @@ -525,13 +529,13 @@ def permute_left_option_groups(l): (1, 2, 3) """ yield tuple() - accumulator = [] + accumulator: list[Parameter] = [] for group in reversed(l): accumulator = list(group) + accumulator yield tuple(accumulator) -def permute_right_option_groups(l): +def permute_right_option_groups(l: ParamGroupSequence) -> Iterator[ParamGroupTuple]: """ Given [(1,), (2,), (3,)], should yield: () @@ -540,13 +544,17 @@ def permute_right_option_groups(l): (1, 2, 3) """ yield tuple() - accumulator = [] + accumulator: list[Parameter] = [] for group in l: accumulator.extend(group) yield tuple(accumulator) -def permute_optional_groups(left, required, right): +def permute_optional_groups( + left: ParamGroupSequence, + required: Iterable[Parameter], + right: ParamGroupSequence +) -> tuple[ParamGroupTuple, ...]: """ Generator function that computes the set of acceptable argument lists for the provided iterables of @@ -561,7 +569,7 @@ def permute_optional_groups(left, required, right): if left: raise ValueError("required is empty but left is not") - accumulator = [] + accumulator: list[ParamGroupTuple] = [] counts = set() for r in permute_right_option_groups(right): for l in permute_left_option_groups(left): |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Argh, the names of those type aliases are quite bad actually. (Again, this shows the peril of type aliases!) Maybe this instead: --- a/Tools/clinic/clinic.py +++ b/Tools/clinic/clinic.py @@ -28,7 +28,7 @@ import textwrap import traceback -from collections.abc import Callable +from collections.abc import Callable, Iterable, Iterator, Sequence from types import FunctionType, NoneType from typing import ( Any, @@ -516,7 +516,11 @@ class PythonLanguage(Language): checksum_line = "#/*[{dsl_name} end generated code: {arguments}]*/" -def permute_left_option_groups(l): +SequenceOfParameterGroups = Sequence[Iterable["Parameter"]] +ParameterTuple = tuple["Parameter", ...] + + +def permute_left_option_groups(l: SequenceOfParameterGroups) -> Iterator[ParameterTuple]: """ Given [(1,), (2,), (3,)], should yield: () @@ -525,13 +529,13 @@ def permute_left_option_groups(l): (1, 2, 3) """ yield tuple() - accumulator = [] + accumulator: list[Parameter] = [] for group in reversed(l): accumulator = list(group) + accumulator yield tuple(accumulator) -def permute_right_option_groups(l): +def permute_right_option_groups(l: SequenceOfParameterGroups) -> Iterator[ParameterTuple]: """ Given [(1,), (2,), (3,)], should yield: () @@ -540,13 +544,17 @@ def permute_right_option_groups(l): (1, 2, 3) """ yield tuple() - accumulator = [] + accumulator: list[Parameter] = [] for group in l: accumulator.extend(group) yield tuple(accumulator) -def permute_optional_groups(left, required, right): +def permute_optional_groups( + left: SequenceOfParameterGroups, + required: Iterable[Parameter], + right: SequenceOfParameterGroups +) -> tuple[ParameterTuple, ...]: """ Generator function that computes the set of acceptable argument lists for the provided iterables of @@ -561,7 +569,7 @@ def permute_optional_groups(left, required, right): if left: raise ValueError("required is empty but left is not") - accumulator = [] + accumulator: list[ParameterTuple] = [] counts = set() for r in permute_right_option_groups(right): for l in permute_left_option_groups(left): |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
- Use a single alias: ParamTuple - Ditch generators, use Iterator - Use Sequence iso. layers of Iterable/Reversible/etc.
|
I did a variant with only one (IMO slightly better named) alias ;) Thank you so much! I'm never found out what the |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
The |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Co-authored-by: Alex Waygood <[email protected]>
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Now we're there ;)
This was a very tricky one!
Sorry, something went wrong.
There was an error while loading. Please reload this page.
* main: (39 commits) pythongh-102542 Remove unused bytes object and bytes slicing (python#106433) Clarify state of CancelledError in doc (python#106453) pythongh-64595: Fix regression in file write logic in Argument Clinic (python#106449) pythongh-104683: Rename Lib/test/clinic.test as Lib/test/clinic.test.c (python#106443) tp_flags docs: fix indentation (python#106420) pythongh-104050: Partially annotate Argument Clinic CLanguage class (python#106437) pythongh-106368: Add tests for formatting helpers in Argument Clinic (python#106415) pythongh-104050: Annotate Argument Clinic parameter permutation helpers (python#106431) pythongh-104050: Annotate toplevel functions in clinic.py (python#106435) pythongh-106320: Fix specialize.c compilation by including pycore_pylifecycle.h (python#106434) Add some codeowners for `Tools/clinic/` (python#106430) pythongh-106217: Truncate the issue body size of `new-bugs-announce-notifier` (python#106423) pythongh-61215: Rename `wait_until_any_call` to `wait_until_any_call_with` (python#106414) pythongh-106162: array: suppress warning in test_array (python#106404) pythongh-106320: Remove _PyInterpreterState_HasFeature() (python#106425) pythonGH-106360: Support very basic superblock introspection (python#106422) pythongh-106406: Fix _Py_IsInterpreterFinalizing() in _winapi.c (python#106408) pythongh-106396: Special-case empty format spec to gen empty JoinedStr node (python#106401) pythongh-106368: Add tests for permutation helpers in Argument Clinic (python#106407) pythonGH-106008: Fix refleak when peepholing `None` comparisons (python#106367) ...
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.