Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added buttons for pre-programmed animations(like flips) and radio button to use leap motion as an input device. #10

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

KarolHausman
Copy link

We will release the ROS driver and additional node for Leap to work with the Ardrone soon.

@KarolHausman
Copy link
Author

Please change nh param called joystick_type to 1 to be able to use ps3 joystick. If you set it to 2 then it is logitech joystick. It is set to 2 by default.

@lesire
Copy link
Member

lesire commented Apr 17, 2014

Hello,
Could you split this pull request into:

  1. an improvement of joystick management; should be good if you could rely on a parameter file to define the specific commands of each joystick/pilot use
  2. an enhancement with support of Leap

@KarolHausman
Copy link
Author

Hi,

it's a good idea, but unfortunately, I won't have time to do it now...
Would you mind helping me with that?
On Apr 17, 2014 5:23 AM, "Charles Lesire-Cabaniols" <
[email protected]> wrote:

Hello,
Could you split this pull request into:

  1. an improvement of joystick management; should be good if you could rely
    on a parameter file to define the specific commands of each joystick/pilot
    use
  2. an enhancement with support of Leap


Reply to this email directly or view it on GitHubhttps://github.com//pull/10#issuecomment-40708086
.

@lesire
Copy link
Member

lesire commented Apr 17, 2014

It's not a bug, but just an enhancement, so it's not for tomorrow ;)
I won't have time until a couple of days...

@KarolHausman
Copy link
Author

OK, I'll work on it as soon as I have some spare cycles. Feel free to do it
if you have time first :)
On Apr 17, 2014 9:10 AM, "Charles Lesire-Cabaniols" <
[email protected]> wrote:

It's not a bug, but just an enhancement, so it's not for tomorrow ;)
I won't have time until a couple of days...


Reply to this email directly or view it on GitHubhttps://github.com//pull/10#issuecomment-40731787
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants