Skip to content

Commit e2c4f90

Browse files
Copilotdorkmo
andcommitted
Address PR review comments: add timestamp field to CalibrationPoint and document obsolete functions
Co-authored-by: dorkmo <[email protected]>
1 parent c1c7696 commit e2c4f90

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

TankAlarm-092025-Client-Hologram/TankAlarm-092025-Client-Hologram.ino

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,7 @@ int largeDecreaseWaitHours = 2;
224224
struct CalibrationPoint {
225225
float sensorValue; // Raw sensor reading (voltage, current, etc.)
226226
float actualHeight; // Actual measured height in inches
227+
String timestamp; // Timestamp of calibration point (for tracking/debugging)
227228
};
228229

229230
// Multi-site / Multi-tank support - all tanks configured via TANKA, TANKB, etc.
@@ -2230,6 +2231,13 @@ void sendPowerFailureNotificationToServer() {
22302231

22312232
// ========== HEIGHT CALIBRATION FUNCTIONS (OBSOLETE - replaced by per-tank calibration in .cfg files) ==========
22322233

2234+
// NOTE: The following obsolete functions reference deleted global variables:
2235+
// - calibrationDataLoaded (bool)
2236+
// - numCalibrationPoints (int)
2237+
// - calibrationPoints[] (CalibrationPoint array)
2238+
// - CALIBRATION_FILE_NAME (constant)
2239+
// These would need to be restored if these functions are ever uncommented.
2240+
22332241
// Obsolete - Load calibration data from SD card
22342242
/*
22352243
void loadCalibrationData() {

0 commit comments

Comments
 (0)