Skip to content

benchmark: q4 ground truth - #101

Closed
GnaneshGnani wants to merge 28 commits into
debs_q3from
debs_q4
Closed

GnaneshGnani wants to merge 28 commits into
debs_q3from
debs_q4

Conversation

@GnaneshGnani

@GnaneshGnani GnaneshGnani commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Q4 — Price range hi/lo (DDSketch p0/p100)

Adds ground truth and 60-minute accuracy benchmark for Q4: per-symbol daily price range (min/max).

Benchmark results

Setup: Day 08-11-21, data_filtered, accuracy_sla=0.01, 12 × 5-min windows
Sketch: DDSketch, relative_accuracy=0.01, quantiles [0.0, 1.0]
Metric: fraction of symbols where both sketch p0 and p100 have ≤ 2% relative error.

Windows Pass rate Verdict
12 (09:00–10:00 CET) 12 / 12 ✅ Perfect — all windows pass

DDSketch p0/p100 (min/max) are essentially exact on this data. First window 0.9997, all subsequent windows 1.0000. This query passes with the widest margin of any evaluated query.

Cumulative: Q1: 11/12 ⚠️ | Q3: not run | Q4: 12/12 ✅

See results/benchmarking.md for per-window breakdown.

- ground_truth/q4.py: add numpy import
- compare.py: frac_hilo_lt_2pct (p0 vs low, p100 vs high)
- run.py: default batch-size 50
- QUERY_PORTING_GUIDE.md: Q4 documentation
The guide stays on disk as an untracked local file.
- ground_truth/q4.py: add numpy import
- compare.py: frac_hilo_lt_2pct (p0 vs low, p100 vs high)
- run.py: default batch-size 50
- QUERY_PORTING_GUIDE.md: Q4 documentation
The guide stays on disk as an untracked local file.
@zzylol

zzylol commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Closing — this branch is conflicting with main (badly diverged, ~32k-line deletions) and predates the consolidated multi-sketch + latency eval harness merged in #494, which redid this ground-truth work. Please reopen and rebase if any ground truth here is still needed.

@zzylol zzylol closed this Jun 14, 2026
@zzylol
zzylol deleted the debs_q4 branch July 17, 2026 20:07
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.

2 participants