experiments: context-tier A/B harness + findings (interface-vs-model reproduction) + ops-context server - #26
Open
shyan-ai wants to merge 1 commit into
Open
Conversation
…xt server Follow-up to PR devrev#10 — the second half of the contribution package, raised to make the placement discussion concrete. Proposes experiments/ as a home for community reproductions. Canary embedded in every file; make validate passes; no data or task changes.
shyan-ai
requested review from
jonmaaku-rev,
mira-mohammad,
nimit2801,
roycabezas and
sumisunz
as code owners
August 6, 2026 06:15
Contributor
|
Thanks for the PR. First-touch read: this is an experiments/community reproduction package, not a leaderboard submission, task change, dataset change, or dependency bump. It adds an interface A/B harness, findings writeup, config variants, and an ops-context MCP server under Checks: GitHub reports no checks on this branch yet; the PR body says Main risk: the new experiment server and findings may be read as benchmark-endorsed results unless placement, reproducibility steps, and caveats stay very explicit. Maintainer verification points:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #10 and the findings comment there — the second half of the package, raised as a concrete proposal so the placement discussion has something to review. Happy to relocate, split, or trim per maintainer preference; the structure below is my best guess at your lanes, not a claim about them.
What this adds (no data changes, no task changes):
experiments/interface-ab/ops-context/— the context-tier MCP server from the A/B: three layers over the existing mock servers — a declarative linkage map, curated governed views (exclusions applied at the data layer, MSA thresholds pre-joined), and a context fingerprint (per-response provenance: which records/documents an answer consumed). It composes like the stock mocks (owndocker-compose.yaml, port 8014) and speaks to the same dataset. I'd have placed it besidecrm/pm/file-server, but/mcp-servers/is a regenerated-archive path (gitignored), so it lives inside the experiment dir — folding it into your archive pipeline instead is a one-word review comment away.experiments/interface-ab/— the harness +FINDINGS.mdfrom the 75-trial reproduction of the benchmark's interface-vs-model result (same model, same tasks, same judge; only the interface differs). Narrative version: https://shyanming.substack.com/p/same-model-different-floor. The n=1-per-cell caveat travels with every number.Why
experiments/as a new directory: proposing it as the home for community reproductions — n=1 reruns of each other's results are the cheapest way to harden leaderboard numbers. If you'd rather this live inscripts/,docs/, or stay external and linked, say the word and I'll restructure — this PR exists to turn the "where should this go" question from #10's comment thread into a reviewable diff.Conformance notes: canary string embedded in every new file (
# noqa: E501on those lines);make validatepasses (ruff + repo validation); the runner's setup hints translated from my local substrate tomcp-servers/compose-up.sh(+ the ops-context compose in this dir). The harness references theops-*tasks from #10 but runs against any task directory with the standard layout.