Fix: _local_order divided by fp*fp that underflows to 0 for a tiny nonzero slope (ZeroDivisionError, caught by the sweep harness and charged as a failure - this is what had moved Table 6's order cells after the sign fix). Pinned by a test that fails on the old code. Every sweep file regenerated on the server (run_all_sweeps.sh). verify_article: skip result files that are not grid lists (it crashed on the out-of-sample dicts). Verifier now completes: 224 OK, 57 stale numbers in the paper to update (Table 6 again 7 of 7).
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
+7
-2
@@ -500,8 +500,13 @@ def verify_safeguard_and_tables():
|
||||
if "smoke" in p:
|
||||
continue
|
||||
with open(p, encoding="utf8") as fh:
|
||||
for r in json.load(fh):
|
||||
cells.setdefault(r["label"], r)
|
||||
data = json.load(fh)
|
||||
# only grid files (a list of labelled cells); the out-of-sample results
|
||||
# (sweep_oos.json, sweep_fourth_*.json, ...) are dicts of a different shape
|
||||
if not isinstance(data, list):
|
||||
continue
|
||||
for r in data:
|
||||
cells.setdefault(r["label"], r)
|
||||
for lbl, aps, cha in (("slope.ratio.derived3.credit", 3507, 2997),
|
||||
("pure+shape+slope.ratio.derived3.credit", 3755, 3021),
|
||||
("pure+dissim+slope.ratio.derived3.credit", 3857, 3002),
|
||||
|
||||
Reference in New Issue
Block a user