From e6e1940069024f539ed2416acb8615c782847aa2 Mon Sep 17 00:00:00 2001
From: Ben Gruver <jesusfreke@jesusfreke.com>
Date: Wed, 7 Jan 2026 00:02:04 -0600
Subject: [PATCH] Fix Bosch144 timings (#2242)

A few of the commands from my Durastar remote were being detected as
Coolix protocol, due to the bit mark and zero space timings being
slightly out of the expected range for Bosch144

Fixes #2241
---
 src/ir_Bosch.h         |  4 ++--
 test/ir_Bosch_test.cpp | 52 ++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 54 insertions(+), 2 deletions(-)

diff --git a/src/ir_Bosch.h b/src/ir_Bosch.h
index f79324e1..729e6551 100644
--- a/src/ir_Bosch.h
+++ b/src/ir_Bosch.h
@@ -30,10 +30,10 @@
 
 // Constants
 const uint16_t kBoschHdrMark = 4366;
-const uint16_t kBoschBitMark = 502;
+const uint16_t kBoschBitMark = 456;
 const uint16_t kBoschHdrSpace = 4415;
 const uint16_t kBoschOneSpace = 1645;
-const uint16_t kBoschZeroSpace = 571;
+const uint16_t kBoschZeroSpace = 610;
 const uint16_t kBoschFooterSpace = 5235;
 const uint16_t kBoschFreq = 38000;  // Hz. (Guessing the most common frequency.)
 const uint16_t kBosch144NrOfSections = 3;
diff --git a/test/ir_Bosch_test.cpp b/test/ir_Bosch_test.cpp
index 39c97ba3..791c7773 100644
--- a/test/ir_Bosch_test.cpp
+++ b/test/ir_Bosch_test.cpp
@@ -126,6 +126,58 @@ TEST(TestDecodeBosch144, DurastarExample) {
   ASSERT_TRUE(IRAcUtils::decodeToState(&irsend.capture, &result, &prev));
 }
 
+// This example command has more extreme timings
+TEST(TestDecodeBosch144, DurastarExample61F) {
+  IRsendTest irsend(kGpioUnused);
+  IRrecv irrecv(kGpioUnused);
+
+  // Mode: Heat; Fan: High ; Temp: 61°F
+  uint16_t rawData[299] = {
+      4450, 4345, 590, 1560, 590, 510, 560, 1560, 590, 1560, 590, 510, 565,
+      510, 560, 1565, 590, 510, 560, 515, 560, 1565, 590, 515, 560, 515, 560,
+      1565, 590, 1565, 585, 520, 560, 1570, 560, 515, 560, 515, 555, 1595, 560,
+      1595, 555, 1595, 530, 1620, 530, 1620, 530, 1625, 525, 1625, 525, 1630,
+      520, 555, 525, 550, 525, 550, 525, 550, 525, 550, 550, 525, 550, 530,
+      550, 525, 550, 530, 550, 525, 550, 1605, 520, 1630, 520, 550, 550, 525,
+      550, 1605, 520, 1630, 520, 1630, 525, 1630, 520, 550, 525, 550, 525,
+      1625, 550, 1605, 550, 5200, 4405, 4390, 520, 1630, 520, 555, 520, 1630,
+      520, 1630, 520, 555, 520, 555, 520, 1630, 520, 555, 520, 555, 520, 1635,
+      520, 555, 520, 560, 520, 1635, 515, 1635, 515, 560, 515, 1655, 495, 580,
+      495, 580, 495, 1660, 495, 1660, 495, 1660, 490, 1660, 490, 1660, 490,
+      1660, 490, 1660, 490, 1660, 490, 585, 490, 590, 485, 590, 485, 590, 485,
+      595, 480, 590, 480, 600, 480, 590, 485, 590, 485, 590, 485, 1670, 485,
+      1670, 485, 590, 485, 590, 485, 1670, 480, 1670, 485, 1670, 480, 1670,
+      480, 595, 480, 590, 485, 1670, 480, 1690, 460, 5290, 4320, 4475, 460,
+      1690, 460, 1690, 460, 615, 460, 1690, 460, 615, 460, 1690, 460, 615,
+      460, 1690, 460, 615, 460, 1690, 460, 1690, 460, 615, 460, 615, 460,
+      1690, 460, 615, 460, 615, 460, 615, 460, 620, 460, 1695, 435, 640, 435,
+      640, 435, 640, 435, 640, 435, 640, 435, 640, 440, 640, 435, 665, 410,
+      1740, 410, 665, 410, 665, 410, 665, 410, 1740, 410, 665, 410, 665, 410,
+      670, 410, 665, 410, 665, 410, 665, 410, 670, 405, 670, 405, 670, 410,
+      1745, 405, 1745, 410, 670, 410, 1745, 380, 670, 405, 1770, 385, 690, 385
+  };  // DURASTAR DRAW09F2A
+
+  uint8_t expectedState[18] = {
+      0xB2, 0x4D, 0x3F, 0xC0, 0x0C, 0xF3,
+      0xB2, 0x4D, 0x3F, 0xC0, 0x0C, 0xF3,
+      0xD5, 0x64, 0x20, 0x11, 0x00, 0x6A};
+
+  irsend.begin();
+  irsend.reset();
+
+  irsend.sendRaw(rawData, 299, 38000);
+  irsend.makeDecodeResult();
+  ASSERT_TRUE(irrecv.decode(&irsend.capture));
+  EXPECT_EQ(decode_type_t::BOSCH144, irsend.capture.decode_type);
+  EXPECT_EQ(kBosch144Bits, irsend.capture.bits);
+  EXPECT_STATE_EQ(expectedState, irsend.capture.state, irsend.capture.bits);
+  EXPECT_EQ(
+      "Power: On, Mode: 6 (Heat), Fan: 5 (High), Temp: 61F, Quiet: Off",
+      IRAcUtils::resultAcToString(&irsend.capture));
+  stdAc::state_t result, prev;
+  ASSERT_TRUE(IRAcUtils::decodeToState(&irsend.capture, &result, &prev));
+}
+
 TEST(TestDecodeBosch144, SyntheticSelfDecode) {
   IRsendTest irsend(kGpioUnused);
   IRrecv irrecv(kGpioUnused);