<div dir="ltr">Jon Binney expressed the same opinion as you guys in the tickets and provided a sample script that would go through and update all instances in a code base:<div><br></div><div><a href="https://github.com/ros/ros_comm/issues/346#issuecomment-36440175">https://github.com/ros/ros_comm/issues/346#issuecomment-36440175</a><br>
</div><div><br></div><div>In principle I agree with with tick-tock'ing the api through indigo and j-turtle, but I am concerned about the maintenance burden it might inflict on us (issues and Q&A questions), but as long as we have a clear document to point people at with the rational, ways to fix it, and maybe something like the script Jon made I think we can handle it.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Mar 6, 2014 at 3:20 AM, Paul Mathieu <span dir="ltr"><<a href="mailto:paul.mathieu@pal-robotics.com" target="_blank">paul.mathieu@pal-robotics.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Right now, it appears that the default setting is easily broken for a minority of use cases and non-intuitive to debug. And that would be good to amend. Better to have it working everywhere and optimisable for power use cases. <br>

</div>
<div><br></div><div>Adding some notes on the wiki probably won't do much to notify existing users - typically copy/pasting from nearby code what I do.</div><div><div><br></div></div><div>So back to my earlier question - is setting a queue size expensive in the python implementation? If there isn't a technical weakness there, then I'm all for a) warnings and a migration point sometime in the future - it wouldn't be very costly to mechanically search and destroy all rospy.Publisher instances in a ros workspace. </div>

</div></div></div></blockquote><div><br></div></div><div>I agree with Daniel. Changing the default will break existing code, but if the default is undesirable in a non-negligible amount of cases, then it should be changed.</div>

<div>I would be in favor for a warning in Indigo, and a change of behavior in J-turtle.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Paul </div></font></span></div></div></div>
<br>_______________________________________________<br>
ros-users mailing list<br>
<a href="mailto:ros-users@lists.ros.org">ros-users@lists.ros.org</a><br>
<a href="http://lists.ros.org/mailman/listinfo/ros-users" target="_blank">http://lists.ros.org/mailman/listinfo/ros-users</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>William Woodall<div>ROS Development Team</div><div><a href="mailto:william@osrfoundation.org" target="_blank">william@osrfoundation.org</a></div><div><a href="http://williamjwoodall.com/" target="_blank">http://williamjwoodall.com/</a></div>

</div>