search

PR2 / pr2_common

45 stars 79 forks source link

projected pattern from projector in gazebo hydro appears backwards #227

Open hsu opened 10 years ago

hsu commented 10 years ago

We might need a fix in urdf or there could be a bug in gazebo.

ahendrix commented 10 years ago

This appears unaffected by the pose tag. I'm going to poke and see if it's a bug in the sdformat urdf parser.

ahendrix commented 10 years ago

Submitted a pull request. Not working yet, but getting closer: https://bitbucket.org/osrf/sdformat/pull-request/96/add-rpyxyz-parser-for-projector-pose/diff

furushchev commented 8 years ago

@hsu @ahendrix Is there any proceedings? I've reproduced this problem on indigo.

screenshot from 2015-12-10 15 27 14

aginika commented 8 years ago

ping

furushchev commented 8 years ago

@hsu @ahendrix ping

furushchev commented 8 years ago

@hsu @ahendrix ping or no maintainer of this package now?

trainman419 commented 8 years ago

I'm no longer working on this.

furushchev commented 8 years ago

I fixed gazebo and sent PR. https://bitbucket.org/osrf/gazebo/pull-requests/2152/fix-remove-back-projection/diff#comment-None

furushchev commented 8 years ago

PR is merged.

23pointsNorth commented 8 years ago

Bump. Issue with projector still not fixed in indigo version.

furushchev commented 7 years ago

@23pointsNorth Sorry for very late response. And yes, my pull request that fixes this issue was merged into gazebo5, though gazebo version that we can install by apt on indigo is 2.2.3., where my patch is not yet applied.

furushchev commented 6 years ago

This problem appears again in gazebo7

pr2_gazebo7

Created an issue in bitbucket https://bitbucket.org/osrf/gazebo/issues/2449/back-projection-fixed-in-gazebo5-still

furushchev commented 6 years ago

From the discussion in https://bitbucket.org/osrf/gazebo/issues/2449/back-projection-fixed-in-gazebo5-still, my fix for gazebo5 was ported into gazebo 7.1 (not gazebo 7.0). To resolve this problem, we just have to upgrade gazebo following http://gazebosim.org/tutorials?cat=install&tut=install_ubuntu&ver=7.0#Alternativeinstallation:step-by-step

v4hn commented 6 years ago

Thank you for your efforts @furushchev this bug is pretty stupid to begin with. I really don't understand how they failed to have their patches in 7.0 when they were released in previous versions...

furushchev commented 6 years ago

@v4hn True. They said that it's very hard to get fixes into the upstream Ubuntu gazebo7 package if it's not fixing a security vulnerability. If you (including me) still want to support me, please leave a comment to the original issue (or create a new issue) to tell them the importance of patching the bugfix. :+1:

v4hn commented 6 years ago

It's not that simple. Gazebo is an upstream Ubuntu package by now. And 7.0 was released in a stable ubuntu distribution. SRUs are very hard to get in there and Gazebo 7.0->7.1 is a much bigger (and less important) change than what they consider worthy an SRU.

The Gazebo devs failed here. Now we have to live with it or use OSRF's custom debs, just as they proposed.