in get_type_hints(Appliction), TypeError: <class 'traitlets.traitlets.Dict'> is not a generic class
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 45/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- python
- Bereich
- developer-experience
Rechercherichtung
Beginnen Sie bei traitlets/config/application.py Zeile 403 und überprüfen Sie die Annotation von subcommands zusammen mit traitlets/traitlets.py Zeile 3842. Reproduzieren Sie anschließend den Fehler mit dem bereitgestellten Beispiel get_type_hints(Application). Erledigt ist es, wenn die Annotation den Wert des Instanz-Dictionarys beschreibt, ohne den nicht-generischen Dict-Trait-Typ einzuschließen, und get_type_hints(Application) diesen TypeError nicht mehr auslöst.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
The expression get_type_hints(Application) raises an exception, given the class traitlets.config.application.Application or a subclass.
Minimal Example
file traitlets_debug_00.py
from traitlets.config.application import Application
from typing_extensions import get_type_hints
if __name__ == "__main__":
hints = get_type_hints(Application)
Traceback from __main__:
Traceback (most recent call last):
File ".../traitlets_debug_00.py", line 6, in <module>
hints = get_type_hints(Application)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib64/python3.11/typing.py", line 2305, in get_type_hints
value = _eval_type(value, base_globals, base_locals)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib64/python3.11/typing.py", line 359, in _eval_type
return t._evaluate(globalns, localns, recursive_guard)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib64/python3.11/typing.py", line 857, in _evaluate
eval(self.__forward_code__, globalns, localns),
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "<string>", line 1, in <module>
File "/usr/lib64/python3.11/typing.py", line 344, in inner
return func(*args, **kwds)
^^^^^^^^^^^^^^^^^^^
File "/usr/lib64/python3.11/typing.py", line 1807, in __class_getitem__
_check_generic(cls, params, len(cls.__parameters__))
File "/usr/lib64/python3.11/typing.py", line 271, in _check_generic
raise TypeError(f"{cls} is not a generic class")
TypeError: <class 'traitlets.traitlets.Dict'> is not a generic class
Analysis
in traitlets.traitlets line 3842, the Dict TraitType is defined like as follows:
class Dict(Instance["dict[K, V]"]):
...
In traitlets.config.application, line 403, the type is applied as:
subcommands: dict[str, t.Any] | Dict[str, t.Any] = Dict()
The Dict TraitType has not been defined as a subscriptable type, however.
Patch?
At an instance scope, the value of the subcommands attribute should generally be a Python dict, while at a class scope, the attribute would be represented with a TraitType descriptor, namely an instance of the Dict TraitType.
As per the note about the syntax of the subcommands value in the source code where the attr is defined, the subcommands attr could be defined as follows:
subcommands: dict[str, tuple[Callable, str]] = Dict()
In effect, this would remove the class-scoped Dict TraitType descriptor from the type hint for the instance-scoped subcommands attribute value, also providing some more detail about the syntax of the attribute value.
The initialization of the subcommands descriptor object would not be changed with this definition.
Workarounds?
As one approach, the following can be used to patch the annotation in the base Application class, while providing an Application-like class with the patched type hint:
from traitlets.config.application import Application
from traitlets.traitlets import Dict
from collections.abc import Callable
import typing as t
class ApplicationShim(Application):
subcommands: dict[str, tuple[t.Union[str, Callable[[t.Self], t.Self]], str]] = Dict()
Application.__annotations__['subcommands'] = ApplicationShim.__annotations__['subcommands']
The type hint in this patch is referenced to how the attribute value is applied in Application.initialize_subcommand() with examples for how the subcommands value is defined as a class variable, in the following:
IPython.core.profileapp.ProfileAppjupyter_client.kernelspecapp.KernelSpecAppipykernel.kernelapp.IPKernelAppipyparallel.cluster.app.IPCluster
To apply the patched type hint insofar as may be visible within editor environments for instance, the class ApplicationShim could be used as a base class in lieu of Application. The patch onto the base subcommands annotation would be the main thing, however, insofar as for application with get_type_hints()
Impacts?
This exception is encountered when calling get_type_hints(Application).
This may affect some reflection-oriented user code and could possibly affect type analysis for the Application class
- Vorherrschende Sprache
- Python
- Sterne
- 653
- Forks
- 217
- Ø Merge
- 2 T. 21 Std.
- Gemergte PRs (30 T.)
- 2
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus ipython/traitlets
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 65/100
-
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
good first issue
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
-
documentation
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 65/100
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
Alle Issues in ipython/traitlets
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
stephrobert/dsoxlab#238 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
sublimehq/package_control#1780 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
nwg-piotr/nwg-displays#145 ·