search

robotology / idyntree

Multibody Dynamics Library designed for Free Floating Robots
BSD 3-Clause "New" or "Revised" License
155 stars 65 forks source link

fix pybind11 high-level binds #1128

Closed andreaostuni closed 7 months ago

andreaostuni commented 7 months ago

This PR solves the problem mentioned in #1127

CLAassistant commented 7 months ago

CLA assistant check
All committers have signed the CLA.

traversaro commented 7 months ago

Thanks @andreaostuni ! Just a curiosity, why are you using the pybind11-bindings instead of the swig-based ones? Do you need a release "quick" with this changes?

traversaro commented 7 months ago

fyi @francesco-romano if you have any opinion.

francesco-romano commented 7 months ago

fyi @francesco-romano if you have any opinion.

Opinion on... what?

traversaro commented 7 months ago

fyi @francesco-romano if you have any opinion.

Opinion on... what?

Sorry, on the PR!

andreaostuni commented 7 months ago

Thanks @andreaostuni ! Just a curiosity, why are you using the pybind11-bindings instead of the swig-based ones? Do you need a release "quick" with this changes?

I'm actually new in the world of bindings and I don't know the difference really well and since I had already had pybind I tried with it!

traversaro commented 7 months ago

I see, thanks! Just a word of warning: most of Python code for iDynTree you may find around (or that LLMs models generate) is probably oriented toward the swig bindings of iDynTree.