Skip to content

Commit 3e2208b

Browse files
lmeyerovclaude
andcommitted
docs(gfql): fix methodology list grouping and scope parity claims
Review follow-ups on PR #1772: - engines.rst Methodology: the prior-sweep bullet no longer dangles with a bare colon over sibling bullets; reworded to state the remaining bullets describe that sweep's setup. - performance.rst + engines.rst: scope the verification claims — cross-engine result-row identity applies to the four-engine fast-path/index/scaling numbers; the vs-Neo4j and vs-Kuzu pairs were validated against expected result rows and cross-database value/row-count checks, not cross-engine identity. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Y6dQEcjdazEnzuvuwf73ZL
1 parent c391db0 commit 3e2208b

2 files changed

Lines changed: 13 additions & 7 deletions

File tree

docs/source/gfql/engines.rst

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -658,9 +658,12 @@ Methodology
658658
- **0.58.0 release-tag sweep** (the seeded typed-hop snapshot, resident-index and
659659
flat-scaling numbers, LDBC SNB SF1 / Neo4j 5.26 pairs, and OLAP q8/q9 above): measured
660660
on the ``0.58.0`` release tag on an NVIDIA DGX Spark (GB10), **warm medians over
661-
N=30 runs**, result rows asserted identical across engines before any timing was kept;
662-
the Neo4j pairs ran on the same box, warm.
663-
- Prior bulk sweep (the Orkut / LiveJournal tables and their derived ratios), pre-0.58.0:
661+
N=30 runs**. Four-engine numbers were kept only after result rows were asserted
662+
identical across engines; the cross-database pairs (Neo4j, Kuzu) were validated against
663+
expected result rows and cross-database value/row-count checks. The Neo4j pairs ran on
664+
the same box, warm.
665+
- Prior bulk sweep (the Orkut / LiveJournal tables and their derived ratios), pre-0.58.0 —
666+
the remaining bullets in this section describe that sweep's setup:
664667
- Host: NVIDIA DGX Spark (GB10 Grace-Blackwell, unified memory — the F3 memory-pressure
665668
boundary is partly a property of this box), RAPIDS container
666669
``graphistry/test-rapids-official:26.02-gfql-polars``.

docs/source/gfql/performance.rst

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,9 @@ Engine speedups at a glance
88

99
GFQL runs the **same query** on four interchangeable engines — ``pandas`` (default),
1010
``polars`` (CPU, columnar), ``cudf`` (NVIDIA GPU), and ``polars-gpu`` (GPU) — and returns
11-
**identical results** on each (differential parity is a release gate; every number on this
12-
page was kept only after the result rows were verified identical across engines).
11+
**identical results** on each (differential parity is a release gate; every four-engine
12+
number on this page was kept only after the result rows were verified identical across
13+
engines, and the cross-database pairs were validated against expected result rows).
1314
Unsupported engine/query combinations are declined before execution during validation,
1415
compilation, or planning rather than silently falling back. The biggest, easiest win is one
1516
keyword, **no GPU required**:
@@ -31,8 +32,10 @@ Release-verified numbers (0.58.0)
3132
---------------------------------
3233

3334
All numbers in this section were measured on the **0.58.0 release tag** on an NVIDIA DGX
34-
Spark (GB10), warm medians over N=30 runs, with results asserted identical across engines
35-
before any timing was kept.
35+
Spark (GB10), warm medians over N=30 runs. The four-engine numbers (seeded fast paths,
36+
resident index, scaling) were kept only after the result rows were asserted identical
37+
across engines; the competitor pairs (vs Neo4j, vs Kuzu) were validated against expected
38+
result rows and cross-database value/row-count checks.
3639

3740
Seeded typed-hop fast path
3841
~~~~~~~~~~~~~~~~~~~~~~~~~~

0 commit comments

Comments
 (0)