Skip to content

Make progress reports in event log clickable to navigate to config tree item#1090

Draft
Copilot wants to merge 3 commits into
mainfrom
copilot/make-progress-reports-clickable
Draft

Make progress reports in event log clickable to navigate to config tree item#1090
Copilot wants to merge 3 commits into
mainfrom
copilot/make-progress-reports-clickable

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 7, 2026

Progress messages for long-running operators (connect, impedance_matrix, polygon operations, etc.) were not clickable in the event log, making it hard to identify which config item was being processed.

The event log UI already parses [[path]] markers as clickable links via DmsEventLog.cpp. This change prepends resultHolder.GetBackRefStr() (which produces the [[path]] format) to all operator progress reports.

Changes

  • Dijkstra.cpp — impedance_matrix per-iteration and summary reports
  • Connect.cpp — ConnectPointOperator (via virtual function param), ConnectInfoOperator, FastConnectOperator (direct resultHolder access)
  • BoostPolygon.cpp — UnionPolygon, ProcessNumOper (inflate/deflate/filter), PolygonConnectivityOperator, polygon overlay
  • BoostGeometry.cpp — all buffer operation overrides (via virtual function param)
  • GridDist.cpp — GridDist iteration progress
  • OperPolygon.cpp — point_in_all_polygons progress

Pattern

Where resultHolder is in scope, the prefix is computed directly. Where it isn't (virtual Calculate methods), CharPtr itemRef = "" is threaded through the call chain.

auto itemRef = resultHolder.HasBackRef() ? resultHolder.GetBackRefStr() + " " : SharedStr();

reportF(SeverityTypeID::ST_MajorTrace, "%simpedance_matrix %s %s of %s sources: resulted in %s od-pairs"
    , itemRef.c_str(), actionMsg, ...);

When no back-reference exists, itemRef is empty and the message format is unchanged.

Copilot AI and others added 2 commits April 7, 2026 18:24
…ation

Prepend the config item path (via resultHolder.GetBackRefStr()) to
progress reports in Connect, impedance_matrix (Dijkstra), BoostPolygon,
and BoostGeometry operators. This makes progress messages in the event
log clickable, navigating to the corresponding tree item.

Affects operators: connect, connect_info, impedance_matrix,
polygon union/split/inflate/deflate/filter, buffer, polygon_connectivity.

Agent-Logs-Url: https://github.com/ObjectVision/GeoDMS/sessions/106cc8e3-7ed2-46aa-b400-a1cc3fa1b2b2

Co-authored-by: MaartenHilferink <2284361+MaartenHilferink@users.noreply.github.com>
… BoostPolygon overlap

Extends clickable tree item navigation in the event log to additional
operators: GridDist, point_in_all_polygons, and polygon overlay.

Agent-Logs-Url: https://github.com/ObjectVision/GeoDMS/sessions/106cc8e3-7ed2-46aa-b400-a1cc3fa1b2b2

Co-authored-by: MaartenHilferink <2284361+MaartenHilferink@users.noreply.github.com>
Copilot AI changed the title [WIP] Add clickable progress reports in event log to tree items Make progress reports in event log clickable to navigate to config tree item Apr 7, 2026
Copilot AI requested a review from MaartenHilferink April 7, 2026 18:33
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.

Request: make progress reports in event log clickable to tree item for connect/impedance_matrix

3 participants