Skip to content

Visualizer: clamp acknowledgement to chain tip - #42

Open
endophysics wants to merge 1 commit into
ShieldedLabs:mainfrom
endophysics:fix-viz-ack-height
Open

Visualizer: clamp acknowledgement to chain tip#42
endophysics wants to merge 1 commit into
ShieldedLabs:mainfrom
endophysics:fix-viz-ack-height

Conversation

@endophysics

Copy link
Copy Markdown
Contributor

Just triggered this PARANOID != WRONG: lo 228029 vs ack 455956.

Prevents the visualizer from stalling when its acknowledged block height exceeds the current best-chain tip.

Changes

  • Clamp the visualizer acknowledgement cursor to the current chain tip before fetching and accepting updates.
  • Add regression coverage for both clamped and in-range acknowledgements.

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