The geometry classes have defaults in a bunch of places and are used as defaults in a bunch of other places, so it would be sensible to make the default readable. Color and Color8Bit are simple enough as well.

These should have a repr similar to dataclasses, IMO.

• Wrap AddressBook usage to corresponding wallet/api/python/*.cpp files prefixed by "Py" that use shared pointers instead of raw pointers.

• Add these new mapped classes to pywallet_api modules at wallet/api/python/pywallet_api.cpp

• wallet/api/wallet2_api.h as reference

For more info on why this is needed read : http://pybind11.readthedocs.io/en/stable/advanced/smart_ptrs.html#std-shared

I noticed the following is always included in the generated stubs.

https://github.com/sizmailov/pybind11-stubgen/blob/65f01c7c34c436f85ac59e92136ce034d11a9145/pybind11_stubgen/__init__.py#L651-L654

For example in my case I don't use numpy, so this by default creates invalid stubs. The _Shape doesn't break any, but I also don't need it. Is this a left over or why does it exist? Thanks