On Fri, Jan 17, 2014 at 12:56 AM, mshayganfar <notifications@github.com>wrote:

> In core/agent/java/edu/wpi/always/client/GazeRealizer.java:
>
> >      *
> >      * @return between (inclusive) -1 (agent's right) and +1 (agent's left)
> >      */
> > -    public static float translateToAgentTurnHor (Point p) {
> > -       // NB: Make sure to change translateAgentTurn if change this
>
> Do we need to change translateAgentTurn in this case?
>

Yes, these two functions must be inverse; translateAgentTurn is called 3
places in GazeBehavior, which you could easily verify (yourself!) in
Eclipse.

> How can we check the potential problem?
>

What do you mean by "potential problem"?

> —
> Reply to this email directly or view it on GitHub<https://github.com/always-on/always/pull/196/files#r8953006>
> .
>



-- 
Dr. Charles Rich, Professor, Computer Science Department<http://www.cs.wpi.edu>
Interactive Media and Game Development
Program<http://www.wpi.edu/academics/imgd/>
Robotics Engineering Program <http://www.wpi.edu/academics/robotics/>
Worcester Polytechnic Institute, Fuller Laboratories B25b
100 Institute Road, Worcester, MA 01609-2280

Email: rich@wpi.edu   Phone: 508-831-5945   Fax: 508-831-5776
Home: http://www.cs.wpi.edu/~rich