Re: [ros-users] Can't fix small glitch on actionlib tutorial…

Top Page
Attachments:
Message as email
+ (text/plain)
+ (text/html)
Delete this message
Reply to this message
Author: User discussions
Date:  
To: User discussions
Subject: Re: [ros-users] Can't fix small glitch on actionlib tutorial wiki page
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 <>wrote:

> On Fri, Feb 11, 2011 at 7:17 AM, Patrick Goebel <>
> 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
> >
> > https://code.ros.org/mailman/listinfo/ros-users
> >
> _______________________________________________
> ros-users mailing list
>
> https://code.ros.org/mailman/listinfo/ros-users
>