Gathiyo / arducopter

Automatically exported from code.google.com/p/arducopter
0 stars 0 forks source link

Failsafe wiki´s contradiction! #340

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
About Failsafe wiki for AC!!

Unfortunately the use of THR_FAILSAFE_ACTION as described in the AC failsafe 
wiki is still inconsistent with with regards to what is stated in the mavlink 
parameters table (as pointed out before). 
They directly contradict each other on the effect of using 1 as value for 
THR_FAILSAFE_ACTION. I cannot believe you really intend this contradiction in 
Ardu wikis, it must be a mistake and it causes confusion and uncertainty. 

Also, it would be recommendable that AC failsafe wiki explicitly states how the 
failsafe will work if THR_FAILSAFE_ACTION is 0, which is actually default value 
according to the Mavlink parameter wiki table.

This issue also exist as a post, with links to the wiki´s I am referring to;
http://diydrones.com/xn/detail/705844:Comment:781674

Please comment on this and correct where applicable.
Thanks. / Tomas

Original issue reported on code.google.com by tomas.so...@gmail.com on 11 Feb 2012 at 11:18

GoogleCodeExporter commented 8 years ago
After two recent test flights of 2.3.1 (downloaded from GIT some hours ago), 
APM2, MP1.1.35, 3DR stock frame I have at least sorted out which one wiki 
correctly describes THR_FS_ACTION; The Mavlink parameter wiki is correct (and 
the AC failsafe wiki is not).
Ergo, the AC FS wiki should be corrected to read:
" If you are in Auto and prefer to continue your mission, the parameter 
THR_FS_ACTION should be set to 2.
If you prefer to have the copter RTL even if it is flying under autopilot, set 
THR_FS_ACTION to 1."

Original comment by tomas.so...@gmail.com on 11 Feb 2012 at 3:12

GoogleCodeExporter commented 8 years ago
Please verify that the above corrections still need to be made for 2.5.5 and 
2.6 and I will be happy to modify the Wiki to include it.
I do see that the 2 values are reversed.
And what is the default action if Fail Safe is set to 0? I agree it should be 
included as well.

Original comment by garyrmccray@gmail.com on 19 Jun 2012 at 12:45

GoogleCodeExporter commented 8 years ago
One of the wiki's mentioned is for ArduPlane and not ArduCopter. I believe the 
ArduCopter wiki is correct.

Original comment by Benny.Si...@gmail.com on 20 Jan 2013 at 11:44

GoogleCodeExporter commented 8 years ago
Issue closed. 
If you find this is an error please report it in the new issues list
https://github.com/diydrones/ardupilot/issues?labels=ArduCopter&page=1&state=ope
n

Original comment by Benny.Si...@gmail.com on 20 Jan 2013 at 11:44