fritob/Camper-Monitor
1
0
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Verbindungssturm abstellen, Einbaulage des Sensors einrichtbar

Browse Source 
Zwei Dinge.

Die Kühlbox piepte dauernd und die Bedienung wurde zäh. Ursache war ein
Fehler in der Verbindungslogik: nach einem fehlgeschlagenen Versuch wurde
der Eintrag freigegeben, und das nächste Advertisement löste sofort den
nächsten aus - bei laufendem Scan bis zu einmal je Sekunde. Geräte
quittieren jeden Versuch, Kühlboxen mit einem Piepton. Nebenher wechselte
der Verbindungszustand im selben Takt, was die Oberfläche in eine
Dauerneuzeichnung trieb.

Ein Versuch wird jetzt für eine Weile gesperrt, mit wachsendem Abstand von
fünf bis sechzig Sekunden. Auch nach einem Trennen durch die Gegenseite
wird nicht sofort neu angeklopft - trennt ein Gerät von sich aus, etwa
weil eine andere App verbunden ist, entstünde sonst ein Wechselspiel aus
Verbinden und Trennen. Beim Umkonfigurieren werden die Sperren
zurückgesetzt, damit ein neu eingerichtetes Gerät sofort drankommt.

Dazu entlastet: bei jeder Antwort wurden alle fünf Protokollparser
durchprobiert, auch wenn längst feststand, welches Protokoll gilt. Steht
der Dialekt, läuft nur noch dieser.

Zweitens der Neigungsmesser: je nach Einbaulage meldet er längs und quer
vertauscht oder mit falschem Vorzeichen. Statt die Lage aus einer Liste
raten zu lassen, misst der neue Assistent sie - zweimal kippen, einmal um
jede Achse, und aus der Reaktion ergibt sich die Zuordnung. Schräges oder
zu schwaches Kippen wird erkannt und gemeldet, statt eine zufällige
Zuordnung zu liefern. Die Sitzung führt die Rohwerte des Sensors weiter
mit, weil der Assistent sie unumgerechnet braucht.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
BiasF
2026-08-30 23:33:47 +02:00
co-authored by Claude Opus 5
parent d67b10d4e0
commit 583aa75595
12 changed files with 583 additions and 57 deletions
Download Patch File Download Diff File Expand all files Collapse all files
CamperMonitor/Bluetooth/BMSSession.swift
+77 -50
View File
@@ -428,68 +428,95 @@ final class BMSSession: NSObject {
}
// MARK: - Auswertung
private func consume(_ data: Data) {
lastResponse = data
receivedByteCount += data.count
buffer.append(contentsOf: [UInt8](data))
if buffer.count > 512 { buffer.removeFirst(buffer.count - 512) }
let (fridgeFrames, fridgeRemainder) = AlpicoolProtocol.extractFrames(from: buffer)
if !fridgeFrames.isEmpty {
buffer = fridgeRemainder
adopt(.alpicool)
for frame in fridgeFrames { alpicoolState.apply(frame) }
alpicoolState.zoneMode = fridgeZoneMode
onFridgeState?(alpicoolState)
publish(alpicoolState.snapshot(deviceID: deviceID, rssi: nil),
usable: alpicoolState.hasStatus)
return
// Steht der Dialekt fest, nur noch diesen prüfen. Bei jeder Antwort
// alle fünf Parser durchzugehen belastet die Funk-Queue ohne Nutzen.
switch dialect {
case .alpicool: consumeAlpicool(); return
case .wattCycle: consumeWattCycle(); return
case .jbd: consumeJBD(); return
case .dalyModbus: consumeDalyModbus(); return
case .dalyClassic: consumeDalyClassic(); return
case .unknown: break
}
let (wattFrames, wattRemainder) = WattCycleProtocol.extractFrames(from: buffer)
if !wattFrames.isEmpty {
buffer = wattRemainder
adopt(.wattCycle)
for frame in wattFrames { wattCycleState.apply(frame) }
publish(wattCycleState.snapshot(deviceID: deviceID, rssi: nil),
usable: wattCycleState.hasUsableData)
return
}
if consumeAlpicool() { return }
if consumeWattCycle() { return }
if consumeJBD() { return }
if consumeDalyModbus() { return }
if consumeDalyClassic() { return }
// JBD als Nächstes: Start-, Endbyte und Prüfsumme machen den Rahmen eindeutig.
let (jbdFrames, jbdRemainder) = JBDProtocol.extractFrames(from: buffer)
if !jbdFrames.isEmpty {
buffer = jbdRemainder
adopt(.jbd)
for frame in jbdFrames { jbdState.apply(frame) }
publish(jbdState.snapshot(deviceID: deviceID, rssi: nil), usable: jbdState.hasUsableData)
return
}
if let start = buffer.firstIndex(where: { $0 == 0xD2 }),
let registers = DalyProtocol.parseModbusResponse(Array(buffer[start...])) {
buffer.removeAll()
adopt(.dalyModbus)
dalyState.apply(registers: registers)
publish(dalyState.snapshot(deviceID: deviceID, rssi: nil), usable: dalyState.hasUsableData)
return
}
let (dalyFrames, dalyRemainder) = DalyProtocol.extractA5Frames(from: buffer)
if !dalyFrames.isEmpty {
buffer = dalyRemainder
adopt(.dalyClassic)
for frame in dalyFrames { dalyState.apply(frame) }
publish(dalyState.snapshot(deviceID: deviceID, rssi: nil), usable: dalyState.hasUsableData)
return
}
// Etwas kam an, ließ sich aber nicht zuordnen: für die Diagnose sichtbar
// machen, damit sich das Protokoll nachträglich bestimmen lässt.
// Etwas kam an, ließ sich aber nicht zuordnen: für die Diagnose
// sichtbar machen, damit sich das Protokoll bestimmen lässt.
publishDiagnostics()
}
@discardableResult
private func consumeAlpicool() -> Bool {
let (frames, remainder) = AlpicoolProtocol.extractFrames(from: buffer)
guard !frames.isEmpty else { return false }
buffer = remainder
adopt(.alpicool)
for frame in frames { alpicoolState.apply(frame) }
alpicoolState.zoneMode = fridgeZoneMode
onFridgeState?(alpicoolState)
publish(alpicoolState.snapshot(deviceID: deviceID, rssi: nil),
usable: alpicoolState.hasStatus)
return true
}
@discardableResult
private func consumeWattCycle() -> Bool {
let (frames, remainder) = WattCycleProtocol.extractFrames(from: buffer)
guard !frames.isEmpty else { return false }
buffer = remainder
adopt(.wattCycle)
for frame in frames { wattCycleState.apply(frame) }
publish(wattCycleState.snapshot(deviceID: deviceID, rssi: nil),
usable: wattCycleState.hasUsableData)
return true
}
/// JBD: Start-, Endbyte und Prüfsumme machen den Rahmen eindeutig.
@discardableResult
private func consumeJBD() -> Bool {
let (frames, remainder) = JBDProtocol.extractFrames(from: buffer)
guard !frames.isEmpty else { return false }
buffer = remainder
adopt(.jbd)
for frame in frames { jbdState.apply(frame) }
publish(jbdState.snapshot(deviceID: deviceID, rssi: nil), usable: jbdState.hasUsableData)
return true
}
@discardableResult
private func consumeDalyModbus() -> Bool {
guard let start = buffer.firstIndex(where: { $0 == 0xD2 }),
let registers = DalyProtocol.parseModbusResponse(Array(buffer[start...]))
else { return false }
buffer.removeAll()
adopt(.dalyModbus)
dalyState.apply(registers: registers)
publish(dalyState.snapshot(deviceID: deviceID, rssi: nil), usable: dalyState.hasUsableData)
return true
}
@discardableResult
private func consumeDalyClassic() -> Bool {
let (frames, remainder) = DalyProtocol.extractA5Frames(from: buffer)
guard !frames.isEmpty else { return false }
buffer = remainder
adopt(.dalyClassic)
for frame in frames { dalyState.apply(frame) }
publish(dalyState.snapshot(deviceID: deviceID, rssi: nil), usable: dalyState.hasUsableData)
return true
}
/// Erster verwertbarer Rahmen: Kandidat und Dialekt stehen fest, ab jetzt
/// im normalen Takt abfragen.
private func adopt(_ newDialect: Dialect) {
CamperMonitor/Bluetooth/BluetoothManager.swift
+53 -4
View File
@@ -162,6 +162,7 @@ final class BluetoothManager: NSObject {
let transport: DeviceTransport
let role: DeviceRole
let fridgeZoneMode: FridgeZoneMode
let sensorOrientation: SensorOrientation
let victronKey: [UInt8]?
}
private var managed: [UUID: ManagedDevice] = [:] // Schlüssel: peripheralID
@@ -170,6 +171,17 @@ final class BluetoothManager: NSObject {
private var levelSessions: [UUID: LevelSession] = [:]
private var connectedPeripherals: [UUID: CBPeripheral] = [:]
private var reconnectTimer: DispatchSourceTimer?
/// Frühester Zeitpunkt für den nächsten Verbindungsversuch je Gerät.
///
/// Ohne diese Sperre entsteht ein Verbindungssturm: schlägt ein Versuch
/// fehl, ist der Eintrag wieder frei, und das nächste Advertisement löst
/// sofort den nächsten aus - bei aktivem Scan bis zu einmal je Sekunde.
/// Geräte quittieren jeden Versuch, Kühlboxen etwa mit einem Piepton.
private var nextConnectAttempt: [UUID: Date] = [:]
private var connectFailures: [UUID: Int] = [:]
private let firstRetryDelay: TimeInterval = 5
private let longestRetryDelay: TimeInterval = 60
private var discoveryFlushTimer: DispatchSourceTimer?
private var discovering = false
@@ -245,6 +257,7 @@ final class BluetoothManager: NSObject {
transport: device.role.transport,
role: device.role,
fridgeZoneMode: device.fridgeZoneMode,
sensorOrientation: device.sensorOrientation,
victronKey: store.victronKeyBytes(for: device.id))
}
let known = Set(devices.map(\.id))
@@ -309,6 +322,15 @@ final class BluetoothManager: NSObject {
}
}
/// Nach einer Änderung der Einbaulage aufrufen.
func updateSensorOrientation(for device: ConfiguredDevice) {
let deviceID = device.id, orientation = device.sensorOrientation
queue.async {
guard let session = self.levelSession(for: deviceID) else { return }
session.orientation = orientation
}
}
/// Setzt die aktuelle Lage des Fahrzeugs als neue Null.
func calibrateLevel(for deviceID: UUID) {
queue.async { self.levelSession(for: deviceID)?.calibrate() }
@@ -346,12 +368,15 @@ final class BluetoothManager: NSObject {
levelSessions[peripheralID] = nil
disconnect(peripheralID)
}
// Zoneneinstellung an bestehende Sitzungen weiterreichen.
// Einstellungen an bestehende Sitzungen weiterreichen.
for device in devices {
bmsSessions[device.peripheralID]?.fridgeZoneMode = device.fridgeZoneMode
levelSessions[device.peripheralID]?.orientation = device.sensorOrientation
}
lastHandledAdvertisement.removeAll()
lastDiagnosticsUpdate.removeAll()
nextConnectAttempt.removeAll()
connectFailures.removeAll()
startScanning()
}
@@ -416,6 +441,11 @@ final class BluetoothManager: NSObject {
existing.state == .connected || existing.state == .connecting {
return
}
// Nach einem Fehlschlag eine Weile Ruhe geben, sonst wird das Gerät
// im Sekundentakt angeklopft.
if let earliest = nextConnectAttempt[device.peripheralID], Date() < earliest {
return
}
guard let peripheral = central?
.retrievePeripherals(withIdentifiers: [device.peripheralID]).first else {
publish { self.linkStates[device.id] = .searching }
@@ -426,6 +456,21 @@ final class BluetoothManager: NSObject {
central?.connect(peripheral, options: nil)
}
/// Sperrt weitere Versuche für eine Weile. Der Abstand wächst mit jedem
/// Fehlschlag, damit ein dauerhaft unerreichbares Gerät nicht endlos
/// angeklopft wird.
private func backOff(_ peripheralID: UUID) {
let failures = (connectFailures[peripheralID] ?? 0) + 1
connectFailures[peripheralID] = failures
let delay = min(firstRetryDelay * pow(2, Double(failures - 1)), longestRetryDelay)
nextConnectAttempt[peripheralID] = Date().addingTimeInterval(delay)
}
private func clearBackOff(_ peripheralID: UUID) {
connectFailures[peripheralID] = nil
nextConnectAttempt[peripheralID] = nil
}
/// Verbindungen fallen im Fahrzeug regelmässig weg – deshalb regelmässig
/// nachfassen statt nur auf das Disconnect-Ereignis zu reagieren.
private func scheduleReconnects() {
@@ -623,6 +668,7 @@ extension BluetoothManager: CBCentralManagerDelegate {
func centralManager(_ central: CBCentralManager, didConnect peripheral: CBPeripheral) {
guard let device = managed[peripheral.identifier] else { return }
connectedPeripherals[peripheral.identifier] = peripheral
clearBackOff(peripheral.identifier)
if device.role == .leveling {
let session = LevelSession(
@@ -637,6 +683,7 @@ extension BluetoothManager: CBCentralManagerDelegate {
self?.publish { self?.levelStates[device.id] = state }
}
)
session.orientation = device.sensorOrientation
levelSessions[peripheral.identifier] = session
session.start()
return
@@ -670,6 +717,7 @@ extension BluetoothManager: CBCentralManagerDelegate {
publish { self.linkStates[device.id] = .failed(message) }
}
connectedPeripherals[peripheral.identifier] = nil
backOff(peripheral.identifier)
}
func centralManager(_ central: CBCentralManager,
@@ -683,9 +731,10 @@ extension BluetoothManager: CBCentralManagerDelegate {
if let device = managed[peripheral.identifier] {
publish { self.linkStates[device.id] = .searching }
// Direkt wieder anfragen; iOS stellt die Verbindung her, sobald das
// Gerät wieder in Reichweite ist.
connectIfNeeded(device)
// Nicht sofort wieder anklopfen. Trennt ein Gerät von sich aus -
// etwa weil eine andere App verbunden ist -, entstünde sonst ein
// Wechselspiel aus Verbinden und Trennen.
backOff(peripheral.identifier)
}
}
}
CamperMonitor/Bluetooth/LevelSession.swift
+15 -2
View File
@@ -32,6 +32,10 @@ final class LevelSession: NSObject {
private var calibrateCharacteristic: CBCharacteristic?
private var state = LevelState()
/// Aus den Geräteeinstellungen; rechnet Sensor- in Fahrzeugachsen um.
var orientation = SensorOrientation.identity {
didSet { applyOrientation() }
}
private var pollTimer: DispatchSourceTimer?
/// Nur nötig, solange das Gerät die Werte nicht von selbst schickt.
private var needsPolling = false
@@ -136,6 +140,13 @@ final class LevelSession: NSObject {
pollTimer = timer
}
/// Rechnet die Rohwerte in Fahrzeugachsen um.
private func applyOrientation() {
let corrected = orientation.apply(pitch: state.rawPitch, roll: state.rawRoll)
state.pitch = corrected.pitch
state.roll = corrected.roll
}
private func publish() {
guard state.hasReading else { return }
onStateChange(.live)
@@ -212,9 +223,11 @@ extension LevelSession: CBPeripheralDelegate {
switch characteristic.uuid {
case Self.pitchUUID:
state.pitch = VanAlignProtocol.angle(from: value)
state.rawPitch = VanAlignProtocol.angle(from: value)
applyOrientation()
case Self.rollUUID:
state.roll = VanAlignProtocol.angle(from: value)
state.rawRoll = VanAlignProtocol.angle(from: value)
applyOrientation()
case Self.offsetsUUID:
if let offsets = VanAlignProtocol.offsets(from: value) {
state.pitchOffset = offsets.pitch
CamperMonitor/Bluetooth/VanAlignProtocol.swift
+4
View File
@@ -53,9 +53,13 @@ enum VanAlignProtocol {
/// Zustand des Neigungsmessers.
struct LevelState: Equatable {
/// Längsneigung: positiv bedeutet, das Heck steht höher als die Front.
/// Bereits auf die Einbaulage des Sensors umgerechnet.
var pitch: Double?
/// Querneigung: positiv bedeutet, die rechte Seite steht höher.
var roll: Double?
/// Wie der Sensor selbst meldet – nötig, um die Einbaulage zu bestimmen.
var rawPitch: Double?
var rawRoll: Double?
var pitchOffset: Double?
var rollOffset: Double?