split parameters for logging and tuning
add a flag to enable/disable tuning: default off
add an option for specifying a folder for outputting logs.
Closes: #10
Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I43864768f1dee8672f13288d3cf5a0c87c373aed
diff --git a/pid/zone.cpp b/pid/zone.cpp
index 928aef5..eef4fde 100644
--- a/pid/zone.cpp
+++ b/pid/zone.cpp
@@ -139,7 +139,7 @@
*/
max = std::max(getMinThermalRPMSetpoint(), max);
- if (tuningLoggingEnabled)
+ if (tuningEnabled)
{
/*
* We received no setpoints from thermal sensors.
@@ -217,7 +217,7 @@
* is disabled? I think it's a waste to try and log things even if the
* data is just being dropped though.
*/
- if (tuningLoggingEnabled)
+ if (loggingEnabled)
{
tstamp now = std::chrono::high_resolution_clock::now();
_log << std::chrono::duration_cast<std::chrono::milliseconds>(
@@ -237,13 +237,13 @@
* However, these are the fans, so if I'm not getting updated values
* for them... what should I do?
*/
- if (tuningLoggingEnabled)
+ if (loggingEnabled)
{
_log << "," << r.value;
}
}
- if (tuningLoggingEnabled)
+ if (loggingEnabled)
{
for (const auto& t : _thermalInputs)
{