Menu

#612 fixes for saphconv (for reading Pioneer parameters)

closed-accepted
Player (393)
5
2010-10-04
2010-08-12
No

Attached are some hacks to the saphconv utility in server/codetools/saphconv. (This utility reads in ARIA .p files [used to be Saphira .p files] and generates code that's used by the p2os driver for Pioneer robots).

The fixes mainly deal with parameters that are given with no value in the .p files. It also adds more parameter names to the "know to be strings" set so that it uses char* when declaring and quotes around the value when assigning the values.

I don't really know TCL so there may be a better way to do some of this stuff.

Discussion

  • Rich Mattes

    Rich Mattes - 2010-08-15

    The changes have been made in SVN trunk.
    Thank you for your contribution.

     
  • Rich Mattes

    Rich Mattes - 2010-08-15
    • status: open --> closed-accepted
     
  • Reed Hedges (Adept MobileRobots)

    updated patch, includes changes to saphconv scripts and changes to robot_params.* in p2os driver as well

     
  • Reed Hedges (Adept MobileRobots)

    fixed saphconv script to force any parameter with "ConvFactor" in its name to be a double. (See developers mailing list Sept. 24 2010)

     
  • Reed Hedges (Adept MobileRobots)

    • status: closed-accepted --> open
     
  • Rich Mattes

    Rich Mattes - 2010-09-27

    Applied updated changes to svn trunk. Thanks!

     
  • Rich Mattes

    Rich Mattes - 2010-10-04
    • status: open --> closed-accepted
     

Log in to post a comment.