search

Gathiyo / arducopter

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

Code Cleanup - consolidate the PID libraries #324

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
We currently have many separate PID libraries.  We should only have one:
   AC_PID   <-- recent creation.  seems complete
   AP_PID   <-- simple lib.  does not store values to eeprom.  remove?
   APM_PI   <-- only PI controller.
   PID      <-- should remove

We should probably combine the objects from AC_PID and APM_PI but combine into 
a library called AP_PID.

Original issue reported on code.google.com by rmackay...@gmail.com on 1 Feb 2012 at 12:25

GoogleCodeExporter commented 8 years ago 
This is a great idea. I'm looking to write a self optimizing controller based 
on fuzzy logic and neural networks (Engineering Senior Design project), to 
replace the existing PI feedback. It would be nice to have the existing code 
consolidated to make it easier to work with.

Original comment by lyu...@gmail.com on 2 Feb 2012 at 12:06

GoogleCodeExporter commented 8 years ago 
Closing all issues on the old issues list by marking them WontFix.

If this is still a valid issue please re-raise it on the new GitHub issues 
list: https://github.com/diydrones/ardupilot/issues

Thanks!

Original comment by rmackay...@gmail.com on 21 Jul 2013 at 2:14