#458 Segway RMP driver update

Unstable
closed-accepted
Player (393)
5
2009-08-01
2009-07-22
Rich Mattes
No

We added support for the USB interface that the newer RMP platforms come with. The driver now works fairly well with position2d commands, and reports odometry, main battery voltage, and UI battery voltage. Position3d should work, but we have never tested it.

The added CAN messages might break compatibility with older powerbases that don't have the USB controller board, but we don't have any older units available to test. It shouldn't be too hard to disable the extra pieces if they get in the way for older units.

Discussion

  • Rich Mattes

    Rich Mattes - 2009-07-22

    segwayrmp driver patch

     
  • Rich Mattes

    Rich Mattes - 2009-07-22
    • labels: --> Player
     
  • Rich Mattes

    Rich Mattes - 2009-07-23
    • assigned_to: nobody --> gerkey
    • milestone: --> Unstable
     
  • G Biggs

    G Biggs - 2009-07-28
    • status: open --> closed-accepted
     
  • G Biggs

    G Biggs - 2009-07-28

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

     
  • Toby Collett

    Toby Collett - 2009-07-31
    • status: closed-accepted --> pending-accepted
     
  • Toby Collett

    Toby Collett - 2009-07-31

    On review of the patch the new files usb_packet.h/cc dont have any copyright or license information. Can the submitter please state the copyright holder and license for these two files? Otherwise we will have to revert the patch.

     
  • Toby Collett

    Toby Collett - 2009-07-31
    • assigned_to: gerkey --> thjc
     
  • Rich Mattes

    Rich Mattes - 2009-07-31

    Those files should be GPL like the rest of the driver. The authors are Eric Grele and Goutham Mallapragda.

     
  • Rich Mattes

    Rich Mattes - 2009-07-31
    • status: pending-accepted --> open-accepted
     
  • Toby Collett

    Toby Collett - 2009-08-01
    • status: open-accepted --> closed-accepted
     
  • Toby Collett

    Toby Collett - 2009-08-01

    files updated to include this info

     

Log in to post a comment.