Repository navigation
M2.17-D: verified SGCB/GoCB status hydration, GOOSE subscribe and scoped Data Model Read - #607
Draft
masarray wants to merge 8 commits into
Conversation
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
M2.17-D — first presentation/integration repair based on October BCUGE screenshots
Stacked on draft #605 (exact SHA
eb44abc9b2ab79db4ccc8ec69faffe9b5ead5f97), not main.Observed: the native GOOSE Sniff workspace receives frames, but the Browser had no explicit GOOSE Subscribe action; Report TrgOps and OptFlds display as separate top-level collapsible sections rather than children of the RCB attributes; Files filename icon/text are flush to the grid cell edge.
Changes:
Subscribe GOOSEbutton on GoCB pages, opening the existingOpenIedGooseSubscriber(device)flow. When capturing, it becomesView GOOSE Capture. No second subscriber, decoder, cache, or MMS GoEna mutation. Operator chooses a NIC through the existing safety mechanism.IedBrowserDetailSectionLayout; preserve exact row identities and typed ARIEC bit interpretation.New verified-SGCB hydration increment: The prior full-discovery priority plan never saw control-block inventory. It now adds only the exact GetNameList-backed
SGCB.NumOfSG,ActSG,LActTmtargets throughIedBrowserControlStateEvidence.ReadTargets, ahead of generic priority leaves, within the same 32-target/10-second bounded serial MMS Read. It does not infer SCL-only status, read editable fields, change the active group, or introduce a second cache/transport. Two new regression tests assert association fencing, deduplication and exclusion of edit commands.GoCB control-state hydration: After full live discovery, the same bounded initial MMS read plan now includes exact live-inventory
GoEna/NdsCom(GO) for up to four GoCBs. This is distinct from passive GOOSE frame reception. Skip stale/current observed values; deny SCL-only or duplicate inventory and every write. Critical LLN0/LPHD Mod/Beh/Health stay ahead of optional control-block reads under the 10-second total budget.Data Model Read scope: A focused DO now shows
Read DOwith a separate compactRead LNaction for the entire currently displayed logical node. The old ambiguousReadlabel could read only four attributes while leaving siblingsNot read, creating a false impression of an unsuccessful full-LN scan. Both buttons call the existing cancellable MMS read code; no new transport.Unresolved physical evidence: BCUGE screenshot shows the selected Browser GoCB
LLN0.BCUwhile GOOSE Sniff showsGOOSE03; wire evidence cannot be borrowed across non-matching control blocks. SGCB attribute partial failures, RCB cross-client state, DataSet analogNot Exposed, and BrowserNot readare not fixed by this PR; require focused protocol/evidence diagnosis, not GUI defaults or unsafe MMS retries. Avoid claiming hardware acceptance.Engine
036e2c00d83a0815b6160147b14ea05975057965and ArdIrec pin unchanged. Keep stacked draft / no main merge.