search

longjie / ps4eye

Apache License 2.0
76 stars 38 forks source link

adding set_camera_info service to both cameras #6

Closed hyaguchijsk closed 9 years ago

hyaguchijsk commented 9 years ago

camera_calibrationを行う際にset_camera_infoというサービスが必要だったので追加しました。

これにより、例えばこの様な形でキャリブレーションができます。

rosrun camera_calibration cameracalibrator.py --size 7x10 --square 0.025 right:=/stereo/right/image_raw left:=/stereo/left/image_raw right_camera:=/stereo/right left_camera:=/stereo/left
k-okada commented 9 years ago

これはset_camera_infoしたらyamlが書きだされるようになっている？なっていない？ なっていないなら，何もおきないのかな．そういうドライバは結構あって，無いままになっているとおもうんだけど，ちがうかな． このへんは１年前の知識なのでああやしいんだけど．

◉ Kei Okada

On Wed, Feb 11, 2015 at 6:11 PM, Hiroaki Yaguchi notifications@github.com wrote:

camera_calibrationを行う際にset_camera_infoというサービスが必要だったので追加しました。

これにより、例えばこの様な形でキャリブレーションができます。

rosrun camera_calibration cameracalibrator.py --size 7x10 --square 0.025 right:=/stereo/right/image_raw left:=/stereo/left/image_raw right_camera:=/stereo/right left_camera:=/stereo/left

You can view, comment on, or merge this pull request online at:

https://github.com/longjie/ps4eye/pull/6 Commit Summary

  • adding set_camera_info service to both cameras

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/longjie/ps4eye/pull/6.

hyaguchijsk commented 9 years ago

seｔ_camera_infoを呼ぶと， 呼びだした側から送られてきたcamera_infoで上書きするようにしてあります． なので，camera_calibration側でcommitを押すと， 新しいcamera_infoを送り続けるようになります．

yamlファイルを書き出すようにはしていません． 書き出すファイルを指定する要素がなく，どこに書き出すべきかが不明瞭です．

ちなみに，set_camera_infoがないと，camera_calibration側でwait_for_serviceに失敗し，先に進めませんでした．

2015年2月11日 18:40 Kei Okada notifications@github.com:

これはset_camera_infoしたらyamlが書きだされるようになっている？なっていない？ なっていないなら，何もおきないのかな．そういうドライバは結構あって，無いままになっているとおもうんだけど，ちがうかな． このへんは１年前の知識なのでああやしいんだけど．

◉ Kei Okada

On Wed, Feb 11, 2015 at 6:11 PM, Hiroaki Yaguchi notifications@github.com

wrote:

camera_calibrationを行う際にset_camera_infoというサービスが必要だったので追加しました。

これにより、例えばこの様な形でキャリブレーションができます。

rosrun camera_calibration cameracalibrator.py --size 7x10 --square 0.025 right:=/stereo/right/image_raw left:=/stereo/left/image_raw right_camera:=/stereo/right left_camera:=/stereo/left

You can view, comment on, or merge this pull request online at:

https://github.com/longjie/ps4eye/pull/6 Commit Summary

  • adding set_camera_info service to both cameras

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/longjie/ps4eye/pull/6.

— Reply to this email directly or view it on GitHub https://github.com/longjie/ps4eye/pull/6#issuecomment-73854850.

k-okada commented 9 years ago

https://github.com/ros-perception/image_pipeline/blob/indigo/camera_calibration/nodes/cameracalibrator.py#L389

hyaguchijsk commented 9 years ago

rosrun camera_calibration cameracalibrator.py --no-service-check --size 7x10 --square 0.025 right:=/stereo/right/image_raw left:=/stereo/left/image_raw right_camera:=/stereo/right left_camera:=/stereo/left と、--no-service-checkをつけることでこのPReqなしでも動くことを確認しました。