Solution: LP-0017 — Whistleblower#48
Conversation
|
|
Hi! Thank you for your submission. I will review later this week and will try to come back with concrete comment. |
| - [x] **Broadcast dedup**: chronicle keeps a per-CID publish ledger; subscribers (incl. `batch-anchor`) skip duplicates. | ||
| - [x] **Batch tool resume after interruption**: seeded from chain at startup + Waku store-protocol catch-up over a configurable lookback window. | ||
| - [x] **CU benchmarks (1 + 50 CID)**: documented in `README_CHRONICLE_REGISTRY.md` § Compute units (cold n=1 = 2,240 cycles; cold n=50 = 178,646 cycles). | ||
| - [x] **Local LEZ deployment**: chronicle-registry deploys to a local LEZ devnet via `make deploy`; `scripts/setup.sh` does so end-to-end. |
There was a problem hiding this comment.
I think we need to have it tested on the testnet/devnet as per criteria here
https://github.com/logos-co/lambda-prize/blob/master/prizes/LP-0017.md#supportability
have you tried?
| - [x] **Batch tool resume after interruption**: seeded from chain at startup + Waku store-protocol catch-up over a configurable lookback window. | ||
| - [x] **CU benchmarks (1 + 50 CID)**: documented in `README_CHRONICLE_REGISTRY.md` § Compute units (cold n=1 = 2,240 cycles; cold n=50 = 178,646 cycles). | ||
| - [x] **Local LEZ deployment**: chronicle-registry deploys to a local LEZ devnet via `make deploy`; `scripts/setup.sh` does so end-to-end. | ||
| - [ ] **E2E integration tests in CI**: IT contract sketched in `integration-test.toml`; chronicle smoke test on the IT topic; that runs against a real local sequencer; `RISC0_DEV_MODE` left unset so the prover runs in production mode CI runner that runs lez and logoscore environment pending. |
There was a problem hiding this comment.
I am not sure on this step. Were you able to validate E2E?
https://github.com/logos-co/lambda-prize/blob/master/prizes/LP-0017.md#supportability
|
|
||
| Worst-case (n=50 near the 100 KiB account-data cap) is ~288k cycles — well under 1 Mi, ~3 % of the NSSA 32 Mi per-tx budget. | ||
|
|
||
| ### Supportability |
There was a problem hiding this comment.
I can't find in this document mentioning of CI is green on the default branch but it is a criteria listed in https://github.com/logos-co/lambda-prize/blob/master/prizes/LP-0017.md#supportability
| @@ -0,0 +1,141 @@ | |||
| # Solution: LP-0017 — Whistleblower | |||
There was a problem hiding this comment.
overall, in this submission I see you are missing GitHub issues filed for any problems encountered with Logos technology
have you had no problems? was everything smooth?
There was a problem hiding this comment.
this is one of the requirements
https://github.com/logos-co/lambda-prize/blob/master/prizes/LP-0017.md#submission-requirements
|
I left some comments, please, let me know if you have questions Meanwhile I will try to play with your submitted software |
Submission for LP-17. Repo: https://github.com/Thompsonmina/WhistleBlower-Logos- · Demo: https://github.com/Thompsonmina/WhistleBlower-Logos-/releases/download/demo-v1/demo.mp4