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

mavenized #22

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

mavenized #22

wants to merge 20 commits into from

Conversation

ronsmits
Copy link

@ronsmits ronsmits commented Jul 5, 2014

I completely mavenized the project and split it up in sub modules:

  • yadrone-core contains the base classes needed by all other projects
  • yadrone-controlcenter is the UI to fly the drone. This also creates a fat jar that can be started as {{java -jar yadrone-controlcenter-0.3.1-with-dependencies.jar
  • yadrone-paperchase
  • yadrone-tutorial
  • yardrone-android. This compiles without problems I does however compile against a very old version of android. Usually the min-sdk points to the lowest level of android supported and the target-sdk points to the latest sdk available (android L or 20) in this case.

@wiseman
Copy link

wiseman commented Jul 6, 2014

I haven't had a chance to try this yet, but I just wanted to say "Awesome."

jfbenckhuijsen and others added 19 commits July 16, 2014 16:14
the mask was only set in case *listeners.size() > 2. Otherwise, the mask
was always reset. Now, the reset is only performed if the number of
listeners drops to 0, and set if there is one listener present (assuming
it's the first).
commands. In case a sticky commands is present, the commands are sent to
the drone too quickly. In result, the video stream halts (probably due
to CPU overload on the drone). By sleeping for a bit (10 ms for now,
this is just a guessed value, should be < the timeout off course) we
allow the video stream a large enough gap to get the data through
the drone before other initialization is done.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants