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

mavcmd: TOL cmds: TOL from current GPS position - initial commit #125

Closed
wants to merge 1 commit into from

Conversation

TSC21
Copy link
Member

@TSC21 TSC21 commented Aug 17, 2014

Needs verification, since it doesn't always enter the gps callback function. May be a sync issue.

@vooon
Copy link
Member

vooon commented Aug 17, 2014

@TSC21 Sorry, but I do not like this implementation:

  1. not needed TOL double service's
  2. wrong subscriber usage

I will write my implementation tomorrow.

@vooon vooon closed this Aug 17, 2014
@TSC21
Copy link
Member Author

TSC21 commented Aug 17, 2014

No problem :) I'm here to learn also. If you want to give a suggestion I can change it myself no need to you do it all over.
Besides it will be easy for me to learn and modify the mavsetp srv.

vooon added a commit that referenced this pull request Aug 18, 2014
@TSC21 TSC21 deleted the TOL_cmds branch August 18, 2014 10:48
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.

None yet

2 participants