Skip to content

fix: expose complete static DataSet inventory and diagnostics - #182

Merged
masarray merged 9 commits into
mainfrom
fix/dataset-diagnostics-completeness
Aug 15, 2026
Merged

fix: expose complete static DataSet inventory and diagnostics#182
masarray merged 9 commits into
mainfrom
fix/dataset-diagnostics-completeness

Conversation

@masarray

Copy link
Copy Markdown
Owner

Problem

ARSAS could still hide Siemens static DataSet members such as cross-LDevice GGIO object-level FCD entries because the SCL engine projection used a different reference identity from the live DataObject model. Existing synthetic tests did not exercise the actual SCL parser/projection path.

Engine dependency

Pins ARIEC61850 main at 4fc3119be2bb97e0ea3a8d31fb990e6a1a089e60 (PR #77), which canonicalizes parser FCDA references such as IED/LD/GGIO6.CBOpnd [ST] to the live-model identity IEDLD/GGIO6.CBOpnd, while keeping FC separately and never inventing .stVal for object-level FCD members.

ARSAS changes

  • Add static DataSet completeness diagnostics: DataSet count, static members, mandatory inventory, represented count, unresolved primary leaf count, and explicit missing references.
  • Include that block in the generated ARSAS diagnostic report for every IED.
  • Add regression coverage through the actual SCL projection path using an Application/LLN0 DataSet targeting ADD/GGIO6.
  • Verify the mandatory object-level member remains visible in Signal Selection inventory without [ST] suffix or fabricated .stVal.

This is intended to make the Siemens CID failure directly observable: a healthy import should show the full static inventory represented, and any gap now prints the missing DataSet/member reference.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@masarray
masarray merged commit 404abd1 into main Aug 15, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant