s-nakaoka / choreonoid

An integrated graphical robotics application framework
http://choreonoid.org
Other
94 stars 58 forks source link

modify python interface for Body #235

Closed kindsenior closed 4 years ago

kindsenior commented 4 years ago

以下の3つの変更です

あと懸念事項としては,データ保持形式がEigen(Column Major)とnumpy(Row Major)で異なっていた(https://pybind11.readthedocs.io/en/stable/advanced/cast/eigen.html#storage-orders) ので,pythonからアクセスするときだけEigenをRow Majorに変換するようにしていますが,こちらもpythonからJ = np.ndarray((6,6), order='F')などすれば不要ではあるので実装方法は要検討かもしれません.

s-nakaoka commented 4 years ago

修正ありがとうございます。 Eigenの参照渡しや要素の並びの問題についてこれまで把握できていませんでした。参照の件はpybind11のマニュアルの説明に従っているのでしたら大丈夫だと思いますし、要素の並びについてもPython側での記述は少ないほうがよいと思うので、これでよいのではないかと思います。