Skip to content

Commit

Permalink
fix: do not use explorer frames for pings in route health checks (#6942)
Browse files Browse the repository at this point in the history
  • Loading branch information
AlCalzone authored Jun 17, 2024
1 parent 467150f commit 159075e
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions packages/zwave-js/src/lib/node/Node.ts
Original file line number Diff line number Diff line change
Expand Up @@ -184,6 +184,7 @@ import {
SupervisionStatus,
type TXReport,
type TranslatedValueID,
TransmitOptions,
ValueDB,
type ValueID,
ValueMetadata,
Expand Down Expand Up @@ -6526,6 +6527,12 @@ protocol version: ${this.protocolVersion}`;
let failedPingsNode = 0;
let latency = 0;
const pingAPI = this.commandClasses["No Operation"].withOptions({
// Don't change the node status when the ACK is missing. We're likely testing the limits here.
changeNodeStatusOnMissingACK: false,
// Avoid using explorer frames, because they can create a ton of delay
transmitOptions: TransmitOptions.ACK
| TransmitOptions.AutoRoute,
// And remember the transmit report, so we can evaluate it
onTXReport: (report) => {
txReport = report;
},
Expand Down

0 comments on commit 159075e

Please sign in to comment.