summaryrefslogtreecommitdiffstats
path: root/pid/pidcontroller.hpp
Commit message (Collapse)AuthorAgeFilesLines
* Add Hysteresis to pid controllersJames Feist2019-02-051-0/+6
| | | | | | | | | | | | Add hysteresis to pid controllers to lower pwm changes. It is defaulted to 0 so it should be transparent to any controller that choses not to implement it. This is the same pattern used by the stepwise controller. Tested-by: Unit tests passed Change-Id: Ib47114285b0017258b7f77eaf067d310f95a0c60 Signed-off-by: James Feist <james.feist@linux.intel.com>
* Move all floats to doublesPatrick Venture2018-11-111-6/+6
| | | | | | | | | The code was developed initially around a pid loop implemented using floats. Therefore, the code was converting back and forth between double for sensor values as inputs and outputs from this PID loop. Change-Id: I2d2919e1165103040729c9f16bb84fde3dd6b81b Signed-off-by: Patrick Venture <venture@google.com>
* pid: add missing override specifiersPatrick Venture2018-10-301-4/+4
| | | | | | | | | | | | | | | | | | [pid/controller.hpp:18] -> [pid/pidcontroller.hpp:28]: (style) The function 'inputProc' overrides a function in a base class but is not marked with a 'override' specifier. [pid/controller.hpp:20] -> [pid/pidcontroller.hpp:30]: (style) The function 'outputProc' overrides a function in a base class but is not marked with a 'override' specifier. [pid/controller.hpp:22] -> [pid/pidcontroller.hpp:32]: (style) The function 'process' overrides a function in a base class but is not marked with a 'override' specifier. [pid/controller.hpp:24] -> [pid/pidcontroller.hpp:34]: (style) The function 'getID' overrides a function in a base class but is not marked with a 'override' specifier. Change-Id: I29a6bbe644a5f7aad933eb49e7b6f7fdc6e33f19 Signed-off-by: Patrick Venture <venture@google.com>
* style: member functions should be lower camelPatrick Venture2018-10-301-8/+8
| | | | | | | Rename member functions to be lower camel instead of snake case. Change-Id: Ib227fd3dadb6d9607290277205223a4324cd4ce5 Signed-off-by: Patrick Venture <venture@google.com>
* Add stepwise controllerJames Feist2018-09-121-0/+60
This adds the ability to use stepwise curves alongside pid control. This creates a base controller class that pidcontroller and stepwise controller inherit from. Note: Hysteresis to come in follow-on patch Tested-by: Created a stepwise controller and noticed that when it crossed a threshold that it contributed to the pwm setting. Change-Id: I6cf842f80eaccafc905d620970afe91e2092d568 Signed-off-by: James Feist <james.feist@linux.intel.com>
OpenPOWER on IntegriCloud