Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider adding optional headers for abseil #172

Open
gerrymanoim opened this issue Jun 26, 2020 · 3 comments · May be fixed by #178
Open

Consider adding optional headers for abseil #172

gerrymanoim opened this issue Jun 26, 2020 · 3 comments · May be fixed by #178

Comments

@gerrymanoim
Copy link
Contributor

#include <libpy/abseil_support.h> if you were using abseil, but otherwise unused.

Any other libs to support?

@llllllllll
Copy link
Contributor

It might be worth adding support for some of the Boost containers. I think that we should add a section on the docs showing py::dispatch::map_to_object (and some of the other helpers) to add a to_object adapter to arbitrary mapping types.

@gerrymanoim
Copy link
Contributor Author

We should also deal with sparsehash, make it optional and add it to a similar _support.h header.

@llllllllll
Copy link
Contributor

llllllllll commented Jun 26, 2020

Maybe we should have a library_converters dir like: library_converters/abseil.h and library_converters/sparsehash.h.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants