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.

Sign up for GitHub

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

Jump to bottom

Avoid to have different logic between c++ code an python bindings #362

Open

GiulioRomualdi opened this issue Jul 9, 2021 · 0 comments

Member

GiulioRomualdi commented Jul 9, 2021 •

edited

Loading

https://github.com/dic-iit/bipedal-locomotion-framework/blob/1bd88a26849ebc076dce39d928bb489eb381f0e7/bindings/python/ParametersHandler/src/YarpParametersHandler.cpp#L33-L40

This problem was raised by @traversaro

Not a big problem, but inserting actual logic in the pybind11 code instead of leaving the logic on the C++ Side and just keep pybind11 code to contain glue code may be a dangerous path, especially if in the future we amt/hope to generate automatically the pybind11 glue code.

Originally posted by @traversaro in #361 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment