feat(python): Add typing to hvplot plot namespace

[thomasaarholt]

Tagging @MarcoGorelli who added the super nice plotting functionality.

I noticed that I didn't get type hinting on the df.plot namespace, so decided to add it here. Were there any particular reasons you didn't do so?

Note that since we from __future__ import annotations, the return type won't raise a runtime this (without strings) despite hvplot possibly not being installed.

[MarcoGorelli]

hey - hvplot doesn't have a py.typed file, so this type annotation wouldn't really have any effect

[thomasaarholt]

hvplot might not have a py.typed file, but the annotation still gives me autocomplete using pyright 😊 :

Here in VSCode:

Same in helix, also using pyright:

[MarcoGorelli]

ah, nice! should be fine then, do you want to add it for Series.plot too then?

[thomasaarholt]

Done! Thanks, I wasn't aware it existed there too!

Thank you so much for the plotting functionality! It really adds that extra bit of elegance to the whole package!

[MarcoGorelli]

thanks @thomasaarholt !

@hoxbro sorry for the ping, just wanted to check if hvplot.plotting.core.hvPlotTabularPolars can be considered public, and if not, if there's anything else which should be used?

[alexander-beedie]

Joining in the fun 😅 Moved the hvimport import into the TYPE_CHECKING block so it can import directly but can't affect runtime; there's at least some chance that'll result in more success across a wider range of type-checkers as they won't have to indirect the lazy import. Tested locally on PyCharm Professional and can confirm that I see autocomplete in the editor, not just the console 👍

[hoxbro]

@hoxbro sorry for the ping, just wanted to check if hvplot.plotting.core.hvPlotTabularPolars can be considered public, and if not, if there's anything else which should be used?

It can be considered public.

And you are always welcome to ping me 👍

[MarcoGorelli]

thanks all!