Skip to content

Fix Event Log value when legacy report sends CDC struct - #189

Merged
masarray merged 5 commits into
mainfrom
fix/event-log-structured-stval
Aug 16, 2026
Merged

Fix Event Log value when legacy report sends CDC struct#189
masarray merged 5 commits into
mainfrom
fix/event-log-structured-stval

Conversation

@masarray

Copy link
Copy Markdown
Owner

Problem

Field Event Log export shows a raw IEC 61850 CDC structure in Value, e.g.:

Structure(3) {stVal=false, q=Quality{...}, t=BinaryTime(...)}

while the monitored signal / IEC telegram is the Boolean ...stVal leaf and PreviousValue is already scalar (False). This produces transitions such as False → Structure(...).

Root cause

ARSAS currently pins an ARIEC61850 engine commit where generic SPS-style status DataObjects can still arrive through the report path as a rendered structure. The Event Log records the formatted display value, so the raw structure leaks into the CSV.

Fix

  • Add a narrow compatibility normalization in Iec61850ValueFormatter for legacy values whose first named structure field is exactly Boolean stVal=true|false.
  • Normalize those values to the existing Boolean display (True / False).
  • Leave non-stVal structures untouched, avoiding accidental collapse of measurement structures.

Regression coverage

Tests use the same Structure(3) {stVal=false, ...} payload observed in the supplied ARSAS Events CSV, cover both False and True, and verify a {mag=...} structure remains unchanged.

The corresponding source fix is in ARIEC61850 PR #79. This ARSAS guard is intentionally retained for compatibility with older/pinned engine builds.

@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 5e060fc into main Aug 16, 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