Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

flir_ptu Rotation Error #15

Closed
RoboEvangelist opened this issue Feb 17, 2015 · 5 comments
Closed

flir_ptu Rotation Error #15

RoboEvangelist opened this issue Feb 17, 2015 · 5 comments

Comments

@RoboEvangelist
Copy link

Hi,

There is a discrepancy between the rviz/urdf model and current sensor.
If I tell the sensor to move to 0.6 position, the physical sensor moves in the correct direction, but the rviz model moves in the opposity direction. I have issues only with the pan...tilt is fine.

I have a PTU46-70.

Where do I correct this issue, in the URDF, or in the driver.h/cpp?

By the way, that is the default behaviour without me modifying anything on the driver side

Thanks

@mikepurvis
Copy link
Member

Hmm. We've already reversed the pan joint once before (390d6ac), however I'm wondering if 598c46f messed things up. The conversation on the commit indicates that it wasn't checked against the hardware.

If you're willing to fix the URDF, and verify that it does the right thing with your hardware, I will merge a PR.

@RoboEvangelist
Copy link
Author

ok, let me try to fix it

@RoboEvangelist
Copy link
Author

These are the values that work for me:

<joint name="${name}_pan" type="revolute">
  <parent link="${name}_pan_link" />
  <origin xyz="0 0 0.066" rpy="-1.5708 0 0" />
  <child link="${name}_tilt_link" />
  <axis xyz="0 -1 0" rpy="3.14159 0 0" />      <------------------------ These values
  <limit lower="${pan_range}" upper="${-pan_range}"
         effort="${joint_effort}" velocity="${pan_velocity}" />
</joint>

@jeff-o
Copy link

jeff-o commented Aug 6, 2015

RoboEvangelist's fix worked for me.

mikepurvis added a commit that referenced this issue Aug 7, 2015
@mikepurvis
Copy link
Member

Committed in 284151a, please verify.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants