On Wed, Sep 22, 2010 at 5:06 AM, Paul Furgale <span dir="ltr"><<a href="mailto:paul.furgale@utoronto.ca">paul.furgale@utoronto.ca</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

Is it possible to get the same effect by sending a CameraInfo message only<br>
when calibration parameters change?<br></blockquote><div><br>This would be a major design change to the image_pipeline. Sending a CameraInfo on change only introduces a synchronization problem; how does the subscriber synch the new parameters with the correct image? Since CameraInfo is lightweight compared to Image, we decided to just pay the (low) cost of sending it every time.<br>
<br>Certainly your way could be made to work, it just makes the code more complicated and we haven't found it necessary. If you write your own specialized rectification nodelet, of course you could use whatever side channel you want for expensive updates.<br>
<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
> For Diamondback I'm planning to refactor the image_pipeline into nodelets.<br>
</div><br>That sounds great! If I do end up going the route of reproducing the<br>
functionality of stereo_image_proc, I'd like to do that on the first pass.<br>
This will avoid extra work when Diamondback comes out. Is there a roadmap<br>
available for what this will look like?<br></blockquote><div><br>The rough schedule for Diamondback work is at <a href="http://www.ros.org/wiki/diamondback/Planning">http://www.ros.org/wiki/diamondback/Planning</a>. The image_pipeline work should be done by November. At some point I'll draw up a more formal design document, but the breakdown will look something like:<br>
<br>Nodelets for:<br> * Color processing: image_raw -> image, image_color<br> * Rectification: image -> image_rect<br> * Stereo correlation: left/image_rect, right/image_rect -> disparity<br> * Point cloud: disparity -> points, points2<br>
<br>Patrick<br></div></div>