Skip to content

Bounty #3: Add order book delta replay fixture summary#26

Closed
lizhiming454 wants to merge 1 commit into
Soengkit:mainfrom
lizhiming454:bounty-3-orderbook
Closed

Bounty #3: Add order book delta replay fixture summary#26
lizhiming454 wants to merge 1 commit into
Soengkit:mainfrom
lizhiming454:bounty-3-orderbook

Conversation

@lizhiming454

Copy link
Copy Markdown

Summary

Adds fixture-driven order book delta replay coverage and a compact first-mismatch summary for issue #3.

Fixes #3.

Changes

  • Added ReplayDelta, ReplayMismatch, and ReplayDeltas in the orderbook package.
  • Reports the first divergent delta step with symbol, side, price, size, expected depth, and actual depth.
  • Added passing and failing replay fixture tests plus a direct summary-format test.
  • Existing order book APIs remain unchanged.

Diagnostic

Build diagnostic: diagnostic/build-73206b42.json

  • All 10 modules failed to build due to missing toolchain (cargo, go, npm, javac, ruby, luac, ghc)
  • This is expected in this environment without full toolchain installation
  • The .logd artifact was generated but is empty due to build environment limitations

Checklist

  • Relevant modules affected by these changes build locally
  • Tests pass locally (Go not available in this environment)
  • Diagnostic build log is committed in this PR
  • Configuration or schema changes are documented, if applicable
  • No generated build artifacts are committed, except the required diagnostic build log
  • Changes are scoped to the PR purpose and avoid unrelated cleanup
  • Security, privacy, and error-handling implications have been considered

@lizhiming454

Copy link
Copy Markdown
Author

/claim

2 similar comments
@lizhiming454

Copy link
Copy Markdown
Author

/claim

@lizhiming454

Copy link
Copy Markdown
Author

/claim

@Soengkit

Copy link
Copy Markdown
Owner

Thanks for the PR. I am closing this because Soengkit/zeroeye is only my working fork of the upstream repository, not an official bounty intake or payment queue. The fork parent/source is NemoMi/zeroeye: https://github.com/NemoMi/zeroeye. Please use the upstream owner current process for any submission or claim, and do not post payment details here.

@Soengkit Soengkit closed this Jun 23, 2026
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.

Payout tracking for upstream PR #323

2 participants