Driver-gaps ledger: 4 UNCLEAR items need a human verdict (#226, #252, #298, #311) #638

New Issue

2026-10-08T09:22:03-04:00

dohertj2 commented

2026-10-08 09:22:03 -04:00

Four items from the #607 branch ledger (docs/plans/2026-10-08-driver-gaps-branch-ledger.md) could not be classified from code alone: master has something related, but whether it satisfies the original request needs a human (or a fixture) to decide. Each needs a verdict: PRESENT (close) or STRANDED (port).

Source: #607.

Checklist:

#226 AbCip - Native STRING / STRINGnn variant decoding (branch d78a471e). Master native runtime handles built-in STRING (82-char capacity, template-derived capacity; 4e6eed74/#570) but no STRINGnn custom-string-type decode found; human to check against fixtures
#252 AbLegacy - Per-device timeout / retry overrides (branch c292dcc1). Master has per-driver and per-device Timeout (AbLegacyDriverOptions.cs:49,104) but no Retries option found; human to confirm if retries still wanted
#298 S7 - Pre-flight PUT/GET enablement test (branch 64a11ef2). S7Driver.cs:744,1156,1161 maps PUT/GET-disabled errors with an actionable message at runtime, but no explicit preflight probe at init; human decide if sufficient
#311 TwinCAT - Handle-based access with caching (branch b67eb6c8). AdsTwinCATClient has variable-handle addressing (0xF005) for handle-addressed symbols but no confirmed handle cache; human check

Filed 2026-10-08 by the hardware-available issue sweep.

Four items from the #607 branch ledger (`docs/plans/2026-10-08-driver-gaps-branch-ledger.md`) could not be classified from code alone: master has something related, but whether it satisfies the original request needs a human (or a fixture) to decide. Each needs a verdict: PRESENT (close) or STRANDED (port). Source: #607. Checklist: - [ ] #226 AbCip - Native STRING / STRINGnn variant decoding (branch d78a471e). Master native runtime handles built-in STRING (82-char capacity, template-derived capacity; 4e6eed74/#570) but no STRINGnn custom-string-type decode found; human to check against fixtures - [ ] #252 AbLegacy - Per-device timeout / retry overrides (branch c292dcc1). Master has per-driver and per-device Timeout (AbLegacyDriverOptions.cs:49,104) but no Retries option found; human to confirm if retries still wanted - [ ] #298 S7 - Pre-flight PUT/GET enablement test (branch 64a11ef2). S7Driver.cs:744,1156,1161 maps PUT/GET-disabled errors with an actionable message at runtime, but no explicit preflight probe at init; human decide if sufficient - [ ] #311 TwinCAT - Handle-based access with caching (branch b67eb6c8). AdsTwinCATClient has variable-handle addressing (0xF005) for handle-addressed symbols but no confirmed handle cache; human check *Filed 2026-10-08 by the hardware-available issue sweep.*

dohertj2 referenced this issue

2026-10-08 09:24:32 -04:00

92 driver-gap issues (#225–#316) were closed against PRs merged into `auto/driver-gaps`, which never reached master #607

dohertj2 referenced this issue from a commit

2026-10-08 16:09:09 -04:00

fix(s7): numeric TSAP loads in form, shared TSAP conflict error, Test Connect read detects PUT/GET disabled, pin Int32 ms rounding (#632 #638)

dohertj2 commented

2026-10-08 16:34:06 -04:00

Verdicts: #226 PRESENT on master. #252 PRESENT on master. #298 DONE (S7 Test Connect now reads 2 bytes of MB0 so PUT/GET-disabled CPUs fail the probe; in Merge b626-s7 2a41b46b). #311 STRANDED / low value (handle cache) - won't port.

Verdicts: #226 PRESENT on master. #252 PRESENT on master. #298 DONE (S7 Test Connect now reads 2 bytes of MB0 so PUT/GET-disabled CPUs fail the probe; in Merge b626-s7 2a41b46b). #311 STRANDED / low value (handle cache) - won't port.

dohertj2 closed this issue

2026-10-08 16:34:06 -04:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dohertj2/lmxopcua#638