-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Open
Description
I'd like to propose that we:
- move
std::optionalandstd::variantconversion support, which are currently inpybind11/stl.h, to a new file (saypybind11/utility.hsince they are in the STL utility library in C++17). - include
pybind11/utility.hfrompybind11/stl.hso that we maintain full backward compatibility. - update the docs Overview page to mention the new header.
My motivation for this proposal is that the conversion for other containers (set/vector/map) in pybind11/stl.h comes with a major downside, documented in http://pybind11.readthedocs.io/en/stable/advanced/cast/stl.html#automatic-conversion, and I believe that optional/variant can be useful in their own right.
If you like the idea, I'm happy to prepare a pull request. Please let me know. #
Metadata
Metadata
Assignees
Labels
No labels