summaryrefslogtreecommitdiffstats
path: root/pid/stepwisecontroller.hpp
Commit message (Collapse)AuthorAgeFilesLines
* Move all floats to doublesPatrick Venture2018-11-111-4/+4
| | | | | | | | | 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: stepwisecontroller: add missing overridePatrick Venture2018-10-301-1/+1
| | | | | | | | | [pid/controller.hpp:24] -> [pid/stepwisecontroller.hpp:35]: (style) The function 'getID' overrides a function in a base class but is not marked with a 'override' specifier. Change-Id: I27f0853e24f6b25aa50f701ce5fad1ea989b618c Signed-off-by: Patrick Venture <venture@google.com>
* style: member functions should be lower camelPatrick Venture2018-10-301-4/+4
| | | | | | | Rename member functions to be lower camel instead of snake case. Change-Id: Ib227fd3dadb6d9607290277205223a4324cd4ce5 Signed-off-by: Patrick Venture <venture@google.com>
* Add hysteresis to stepwise controllerJames Feist2018-09-211-0/+3
| | | | | | | | Tested-by: Ran on platform monitoring output and wrote unit test Change-Id: I74a1d21544c1a9cb4c1cb26dd4a353cbff0442d0 Signed-off-by: James Feist <james.feist@linux.intel.com>
* Add stepwise controllerJames Feist2018-09-121-0/+52
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