ros-perception / calibration

Provides a toolchain to calibrate sensors and URDF robot models.
http://www.ros.org/wiki/calibration
33 stars 30 forks source link

add calibration_setup_helper program #31

Closed k-okada closed 9 years ago

k-okada commented 10 years ago
<launch>
  <include file="$(find baxter_gazebo)/launch/baxter_world.launch" />
  <anode pkg="baxter_interface" type="joint_trajectory_action_server.py"
        args="--limb both" name="joint_trajectory_action_server" />
  <node pkg="image_proc" type="image_proc" name="camera_rgb_image_proc"
        ns="/camera/rgb" />
  <node pkg="topic_tools" type="relay" args="/robot/joint_states /joint_states"
        name="joint_states_relay" />
  <node pkg="image_view" type="image_view" name="image_view_rect" >
    <remap from="image" to="/camera/rgb/image_rect" />
  </node>

</launch>
(cd src; rosrun calibration_setup_helper calibration_setup_helper.py
        `rospack find baxter_description`/urdf/baxter.urdf
        --base-link 'torso' --arm-end-link 'right_gripper_base'
        --head-end-link 'camera_rgb_frame'
        --arm-controller=/robot/limb/right/command
        --head-controller=/robot/limb/head/command)
rosrun baxter_calibration make_samples.py

use C-c to step

roslaunch baxter_calibration capture_data.launch
rosrun baxter_calibration calibrate_baxter.sh
k-okada commented 10 years ago

you need to include foolowing PRs to run examples

k-okada commented 10 years ago

It may not work well, please do not merge yet

k-okada commented 10 years ago

I think it is now ok to merge

garaemon commented 10 years ago

+1

vrabaud commented 10 years ago

also, please add a REAMD file that contains what you described at the beginning of that bug report. @mikeferguson , any opinion against ? Looks pretty useful to me

mikeferguson commented 10 years ago

I didn't look at it in detail, but it seems fine.

vrabaud commented 9 years ago

please set yourself as an author/maintainer. Installing your script would be more proper with a setup.py (especially to make your code runnable from Python3)

k-okada commented 9 years ago

ok, I have fexed, and also create #37 that include branch to use setup.py. Just to make sure that if se use setup.py , it install the script under global bin instead of package bin. Is it ok?

vrabaud commented 9 years ago

You are totally right, this is not ok :( I was wrong. The setup.py helps dealing with Python 3 but let's ignore that for now. Just update the maintainers/authors and license on this pull request. Sorry for the noise.