Adds stub type definitions to surgepy. Fixes #7362

[KingaJanicka]

I've manually ran pybind11-stubgen surgepy and added the resulting stubs to the surgepy package install routine. This is similar to how pybind11-stubgen author recommends these stubs be used; see sizmailov/pybind11-stubgen#104 (comment).

It's possible there is a better way of accomplishing this, however I am very unfamiliar with skbuild.

[narenratan]

Hi there, Naren here (person who set up the surgepy packaging). This looks great to me! I can successfully pip install surgepy with this branch and the surgepy tests pass.

[baconpaul]

Yes happy to merge

My only question is what do we do if we update the bindings? Can we add that as a comment in the cpp file so we can keep things in synch?

[KingaJanicka]

@baconpaul that sounds like a good idea! I'll leave it to your discretion where to put that. Ideally it would generate the stubs post install but I'm not sure how to do that with skbuild (pybind11-stubgen expects the package to be installed via pip afaik).

[baconpaul]

Really just put a block comment in this file after the boilerplate comme t with what you ran
https://github.com/surge-synthesizer/surge/blob/main/src/surge-python/surgepy.cpp

And add yourself to authors and push and we can merge

Cool thanks!

[KingaJanicka]

@baconpaul have done both, thank you!

[baconpaul]

Awesome one of us can fix up that clang format and merge this for 1.3 this eeek

[mkruselj]

@KingaJanicka You haven't added yourself to AUTHORS file. Please do that as well before we merge. :) Thanks!

[baconpaul]

@KingaJanicka You haven't added yourself to AUTHORS file. Please do that as well before we merge. :) Thanks!

There’s an authors diff in this pr! Did you not read the merged diff maybe?

I will format and merge this this am