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

Support for config acks and multiconfig options (for the cool stuff) #33

Closed
bkw opened this issue Dec 31, 2012 · 4 comments
Closed

Support for config acks and multiconfig options (for the cool stuff) #33

bkw opened this issue Dec 31, 2012 · 4 comments

Comments

@bkw
Copy link
Collaborator

bkw commented Dec 31, 2012

I wrote a gist on why we need to do something about client.config() and what my current plan of attack is here: https://gist.github.com/4416785

Maybe this should go into the new api? It's really time I had a look at it.

@felixge
Copy link
Owner

felixge commented Jan 15, 2013

Had a look at the gist - LGTM!

About new api: Huh, I guess somebody needs to go ahead and finish this. I think we also need a comparability mode for the old api if we push this. Not sure if I will find the time right away. How much hacking time do you have atm @bkw ? : )

@bkw
Copy link
Collaborator Author

bkw commented Jan 15, 2013

Not much at the moment. There might be more time in February though.

@wiseman
Copy link
Collaborator

wiseman commented Oct 6, 2013

I think this would help with #74 too. It seems that to update ephemeris you need to do a client.config('gps:ephemeris_uploaded','RESET', 1) and make sure it's been sent and processed before starting the FTP transfer--without an ACK I think the only alternative is to wait some amount of time and hope it went through.

Once the FTP is done, you need to send another gps:ephemeris_uploaded config to tell the drone to process the ephemeris. My guess is that you want to send that config once and only once, which an ACK would also help with.

I played a little with AT*CTRL to control ACKing, but didn't get it working the way I expected. Does anyone else have the start of code to handle this?

@wiseman
Copy link
Collaborator

wiseman commented Jul 13, 2016

Looks like maybe ROS has a complete config implementation: AutonomyLab/ardrone_autonomy#26

@bkw bkw closed this as not planned Won't fix, can't repro, duplicate, stale Jun 23, 2022
Repository owner deleted a comment from liuruitv Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants