Skip to content

Commit 2436b50

Browse files
luisleo526claude
andcommitted
docs(readme): correct coverage figures and ctest binary count
Ran scripts/coverage.sh fresh: 77/77 tests pass, line coverage 93.26% (was stale at 93.06%), branch coverage 81.07%. The README body text's "~81% line coverage" was a units mislabel of branch coverage, not a second stale line-coverage figure. ctest suite also grew 39->77 binaries (76 C++ + 1 ABI sanity) since the doc was last touched. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent 898a997 commit 2436b50

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<div align="center">
22

3-
<img src=".github/assets/pineforge-banner.jpg" alt="PineScript backtests, deterministic, on your data — v0.8.0 · 93.06% line coverage · 251/252 strict TV parity · 0 engine bugs" width="900">
3+
<img src=".github/assets/pineforge-banner.jpg" alt="PineScript backtests, deterministic, on your data — v0.8.0 · 93.26% line coverage · 251/252 strict TV parity · 0 engine bugs" width="900">
44

55
# PineForge
66
[![CI](https://img.shields.io/github/actions/workflow/status/pineforge-4pass/pineforge-engine/ci.yml?branch=main&label=ci&logo=github)](https://github.com/pineforge-4pass/pineforge-engine/actions)
@@ -189,7 +189,7 @@ This repository ships:
189189
- `libpineforge.a` — the static runtime library
190190
- `<pineforge/pineforge.h>` — the public C ABI (the canonical, stability-pinned consumer surface)
191191
- `<pineforge/*.hpp>` — the internal C++ headers (the PineForge transpiler emits against these; not part of the stability guarantee)
192-
- A 39-binary ctest suite (38 C++ + 1 pure-C ABI sanity test) that runs in CI on every commit (~81% line coverage of `src/` measured via `bash scripts/coverage.sh`)
192+
- A 77-binary ctest suite (76 C++ + 1 pure-C ABI sanity test) that runs in CI on every commit (93.26% line coverage / 81.07% branch coverage of `src/` measured via `bash scripts/coverage.sh`)
193193
- `**corpus/`** (**public git submodule**) — **252 reference strategies** under a single `corpus/validation/` tree. Each folder ships `strategy.pine`, `generated.cpp`, `tv_trades.csv`, and `engine_trades.csv`. Run `bash scripts/run_corpus.sh` after `git submodule update --init corpus`.
194194
- `[benchmarks/](benchmarks/)`**three-way engine comparison** (PineForge ↔ [PyneCore](https://github.com/PyneSys/pynecore)[PineTS](https://github.com/LuxAlgo/PineTS)) on 100 strategies (50 public + 50 promoted corpus probes) and 10 canonical indicators. The harness code and reports live here; **fixtures** (pinned OHLCV, every `strategies/`* folder with TV exports and trade CSVs) ship via the optional **`benchmarks/assets` submodule** — a separate optional **public** submodule (Apache-2.0). With that init’d, `bash benchmarks/run_all.sh` reproduces the headline numbers with zero external API calls. PyneCore Python is official cloud-compiler output (no hand-ports). Headline: PineForge hits canonical *excellent* tier on **50/50** strategies (first 50) vs PyneCore’s 47/50; on the expanded **100-strategy suite (~167,000 TV trades verified)**, PineForge holds **100/100 excellent** vs PyneCore’s 85/100. Median speedup: 162× vs PyneCore across 99 commonly-timed strategies.
195195

@@ -354,7 +354,7 @@ src/ - implementation (~25 .cpp files split by concern)
354354
│ ├── ta_extremes_volume.cpp Highest/Lowest, OBV, AccDist, NVI/PVI/PVT, VWAP, ...
355355
│ └── ta_misc.cpp Linreg, PercentRank, BarsSince, ValueWhen, ...
356356
└── magnifier.cpp / matrix.cpp / session_time.cpp / str_utils.cpp / timeframe.cpp / timezone.cpp / math.cpp
357-
tests/ - 39 ctest binaries (38 C++ + 1 pure-C ABI sanity)
357+
tests/ - 77 ctest binaries (76 C++ + 1 pure-C ABI sanity)
358358
corpus/ - public submodule: 252 strategies; see CONTRIBUTING.md
359359
├── data/ - reference 36k-bar OHLCV feed (Binance ETH/USDT:USDT 15m)
360360
└── CMakeLists.txt - opt-in subproject that compiles every generated.cpp into strategy.so

0 commit comments

Comments
 (0)