search

jsk-ros-pkg / jsk_model_tools

JSK model utilities
https://github.com/jsk-ros-pkg/jsk_model_tools
BSD 3-Clause "New" or "Revised" License
6 stars 29 forks source link

use changes for using glbody in irtgl.l #131

Closed YoheiKakiuchi closed 8 years ago

YoheiKakiuchi commented 9 years ago

merge after irteus released

k-okada commented 9 years ago

why this PR passed without new version of jskeus is not released

k-okada commented 8 years ago

最近このPRをマージしたので、それ以前に作ったモデルと整合性がなくなっているとおもいます。 モデルを作りなおしてみてください

@YoheiKakiuchi コレはどれぐらい副作用が有りますか？　pr2.l なども作り直しですか？

YoheiKakiuchi commented 8 years ago

はい、作り直します。

現象としては、単体では使えているが、クラス定義が書き変わっているので、 古い定義が書いてあるモデルと新しい定義が書いてあるモデルを同時にロードするとエラーになっていということのようです。

YuOhara commented 8 years ago

@YoheiKakiuchi お手すきのときに、pr2のモデルをアップデートしてもらえますでしょうか？

;; error 
1.irteusgl$ (load "package://pr2eus/pr2.l")
(load "package://hrpsys_ros_bridge_tutorials/models/samplerobot.l")
(setq *pr2* (instance pr2-robot :init))
(setq *sr* (instance samplerobot-robot :init))
(objects (list *sr* *pr2*))

t
2.irteusgl$ t
3.irteusgl$ #<pr2-robot #X5897da8 pr2  0.0 0.0 0.0 / 0.0 0.0 0.0>
4.irteusgl$ #<samplerobot-robot #X6c19af8 SampleRobot  0.0 0.0 0.0 / 0.0 0.0 0.0>
5.irteusgl$ ;; (make-irtviewer) executed
/home/leus/ros/indigo_parent/devel/share/euslisp/jskeus/eus/Linux64/bin/irteusgl 0 error: unbound variable glvertices in (if glvertices (progn (send glvertices :draw vwr)))
6.E1-irteusgl$ nil
k-okada commented 8 years ago

ちゃんと中身を理解していませんが、バックワードコンパチビリティを確保する方向で進められるか、検討してください

モデルを作り直せば解決する、は、あまり解決にはならず、ユーザにそういう作業を強いる変更はスジが良く無いです