-
Notifications
You must be signed in to change notification settings - Fork 226
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
Control configuration changes not reflected on drone? #24
Comments
Thanks for debugging this issue @mikehamer . I personally found this issue a couple of month ago, but I have not looked into it with details. Regarding the "ARDrone SDK 2 Developers Guide" (Multiconfiguration) Section (in addition to what you said), there are four types of configurations:
First time that I implemented the This is my suggested roadmap to fix this problem:
This is what have been found so-far:
|
Fixed in my pull-request #26 |
Fix Snafu: Forgot to uncomment
It appears that changes in control parameters (and other CAT_USER) settings do not affect the drone.
When running ardrone_driver, the following is printed:
Investigating the drone's configuration files (see next code snippet) shows that the CAT_SESSION values (for example detect_type, detections_select_h, detections_select_v_hsync) are correctly set (as implied by the driver output), but custom CAT_USER settings are not set. The type of each parameter can be read in
config_keys.h
.CAT_SESSION parameters correctly set
But CAT_USER not?
There are no user profiles created by the application.
Further debugging by setting
#define _GENERAL_NAVDATA_DEBUG (1)
inardrone_general_navdata.c
produces the following output on driver startup:The driver seems to use the default user profile, which is not updated according to the set ros parameters.
Changing the
euler_angle_max
parameter to 30º instead of the default 12º and running the drone in a vicon system shows no change in drone behavior with the increased parameter value (this is also visually apparent).Perhaps I am doing something incorrectly, in which case please let me know! If however this is an issue, I am happy to help debug – let me know what I can do!
Mike
The text was updated successfully, but these errors were encountered: