Skip to content

Conversation

@xchang1
Copy link
Contributor

@xchang1 xchang1 commented Oct 26, 2025

This updates libhandlegraph to make equality of net_handle_t's show up in the python bindings.
Update of #190

@xchang1
Copy link
Contributor Author

xchang1 commented Oct 29, 2025

This also adds stuff from #223

@adamnovak
Copy link
Member

It looks like the way #223 is trying to generate bindings for std::unordered_set can't work, because those bindings end up being specific to the STL implementation thy were generated on (mentioning types like std::__detail::_Node_iterator<std::string, true, true>) and can't be committed to the repo and expected to build on other platforms.

If we can't convince Binder to generate bindings that work for everyone's std::unordered_set, we might need to try Binder's --include-pybind11-stl option from https://cppbinder.readthedocs.io/en/latest/config.html to get it to do some magic for converting these containers to/from Python containers, instead of binding them normally.

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 this pull request may close these issues.

4 participants