Thanks Jeremy and Eitan!  It was rather minor, but I thought I should
report it.  Looks great now.

--patrick

On 2/11/2011 9:24 AM, Eitan Marder-Eppstein wrote:
> Ah yes, I was rushing through changing the tutorials and just blew off
> the code-ref sections. Things should be fixed across all the actionlib
> tutorials now... this wasn't the only one I screwed with.
>
> Sorry for any confusion this caused.
>
> Hope all is well,
>
> Eitan
>
> On Fri, Feb 11, 2011 at 9:12 AM, Jeremy Leibs <leibs@willowgarage.com
> <mailto:leibs@willowgarage.com>> wrote:
>
>     On Fri, Feb 11, 2011 at 7:17 AM, Patrick Goebel
>     <patrick@pirobot.org <mailto:patrick@pirobot.org>> wrote:
>     > I noticed a small glitch on the actionlib tutorial page for
>     writing a
>     > Python server:
>     >
>     >
>     http://www.ros.org/wiki/actionlib_tutorials/Tutorials/Writing%20a%20Simple%20Action%20Server%20using%20the%20Execute%20Callback%20%28Python%29
>     >
>     > The code breakdown for lines 40-45 is interrupted with the error:
>     >
>     > ERROR: unindent does not match any outer indentation level
>     >
>     > I saw that the CodeRef for that block was for lines 40-42 so I
>     changed
>     > it to 40-45 but that did not fix the error so at this point I
>     need to
>     > punt.  (The Python indentation looks fine to my eye but maybe
>     someone
>     > else can catch it.)
>
>     The issue is that line 40-42 were:
>
>            break
>        self._feedback.sequence.append(self._feedback.sequence[i] +
>     self._feedback.sequence[i-1])
>        # publish the feedback
>
>     I'm guessing the python code parser being applied to the range of
>     values still wants it to be a valid python block, which this isn't.
>
>     Changing it to 41-43 fixed the problem.  However, looking at all the
>     other code blocks, many of them now have the off-by-one error.
>
>     I looked through a change log, and it looks like Eitan added:
>
>     self._as.start()
>
>     To line 18 of the referenced code block, but didn't update any of the
>     coderefs.  Any CodeRef referencing a number greater than 18 will need
>     to be incremented by 1.  Also, I'm sure one of the sections referenced
>     the interval including 18 and the doc probably needs to be changed to
>     include calling self._as.start()
>
>
>     >
>     > --patrick
>     >
>     > _______________________________________________
>     > ros-users mailing list
>     > ros-users@code.ros.org <mailto:ros-users@code.ros.org>
>     > https://code.ros.org/mailman/listinfo/ros-users
>     >
>     _______________________________________________
>     ros-users mailing list
>     ros-users@code.ros.org <mailto:ros-users@code.ros.org>
>     https://code.ros.org/mailman/listinfo/ros-users
>
>
>
> _______________________________________________
> ros-users mailing list
> ros-users@code.ros.org
> https://code.ros.org/mailman/listinfo/ros-users
>