dohertj2/lmxopcua
1
0
Fork 0
Code Issues 14 Pull Requests Actions Packages Projects Releases Wiki Activity

Clarify S7 counter evidence and fixture limits #715

Merged
dohertj2 merged 1 commits from docs/s7-counter-evidence into master 2026-10-13 11:54:59 -04:00
Conversation 0 Commits 1 Files Changed 3
+78 -25
dohertj2 commented 2026-10-13 11:45:22 -04:00
Owner
Copy Link
Copy Source

The S7 counter docs previously inferred C-area wire encoding from CPU family and presented synthetic raw4711/31337 values as current counts and hardware-ready evidence. State the actual raw big-endian decoder contract, distinguish DB/IEC reads, and document the independent PLC/raw-response/model evidence still needed under #616. Qualify the current S71500 fixture’s limits.

Validation: independent review clear; exact non-comment C# text identity; XML documentation parses; diff whitespace check passed. Decoder/config/transport/assertions/skip strings unchanged. No build, test run or live call was needed for this comments/docs-only correction. #616 remains open.

Resolves #713.

The S7 counter docs previously inferred C-area wire encoding from CPU family and presented synthetic raw4711/31337 values as current counts and hardware-ready evidence. State the actual raw big-endian decoder contract, distinguish DB/IEC reads, and document the independent PLC/raw-response/model evidence still needed under #616. Qualify the current S71500 fixture’s limits. Validation: independent review clear; exact non-comment C# text identity; XML documentation parses; diff whitespace check passed. Decoder/config/transport/assertions/skip strings unchanged. No build, test run or live call was needed for this comments/docs-only correction. #616 remains open. Resolves #713.
dohertj2 marked the pull request as ready for review 2026-10-13 11:54:46 -04:00
dohertj2 changed target branch from fix/bootstrap-and-boundary-test-fixtures to master 2026-10-13 11:54:46 -04:00
dohertj2 added 1 commit 2026-10-13 11:54:47 -04:00
docs: clarify S7 counter evidence and fixture limits cd50db136e
dohertj2 merged commit 6f0674296b into master 2026-10-13 11:54:59 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
auto-managed
queue management
 cross-driver
queue management
 driver/abcip
queue management
 driver/ablegacy
queue management
 driver/focas
queue management
 driver/mqtt
MQTT / Sparkplug B broker-subscribe driver
 driver/opcuaclient
queue management
 driver/s7
queue management
 driver/twincat
queue management
 phase/1
queue management
 phase/2
queue management
 phase/3
queue management
 phase/4
queue management
 phase/5
queue management
 phase/6
queue management
 queue/blocked
queue management
 queue/done
queue management
 queue/failed
queue management
 queue/in-progress
queue management
 queue/queued
queue management
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dohertj2 (dohertj2)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dohertj2/lmxopcua#715
Delete Branch "docs/s7-counter-evidence"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?