Bump EigenScript pin v0.32.0 -> v0.33.0 - #40
Merged
Conversation
v0.33.0 is the silent-wrong-answer release: the plausible-but-wrong-at-rc=0 sweep (regex_match #629, json_decode #628, four stdlib math bugs #638-#641, import top-level scope #673, the listcomp/local double binding #633/#642, report_value #674), plus DEFLATE codecs (#684), clock_unix (#683), four C-backed list ops (#543/#544), and kill -USR1 observer dumps (#660). This repo's suite was run locally against the v0.33.0 runtime BEFORE this bump and passed unchanged — no code change needed here. (The same sweep did find real breakage in ouroboros, iLambdaAi and EigenRegex, fixed in their own PRs: EigenScript #630 widened OP_LINE's operand, which is a bytecode ABI break for external producers, and #629 changed regex_match's shape for non-participating groups.) Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates Tidepool’s pinned EigenScript runtime to v0.33.0, keeping the devcontainer/CI environment aligned with the latest upstream release and its correctness fixes.
Changes:
- Bump the devcontainer EigenScript ref from v0.32.0 to v0.33.0.
- Update developer documentation to reflect the new pin.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
CLAUDE.md |
Updates the stated CI/devcontainer EigenScript pin to v0.33.0. |
.devcontainer/Dockerfile |
Bumps EIGS_REF to v0.33.0 (the actual runtime pin used by devcontainer/CI). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
27
to
+30
| # EIGS_REF is a tag/SHA (fetched directly, so it stays reproducible). Pinned to | ||
| # v0.19.0 — the release that ships the flat-buffer tensors (shaped VAL_BUFFER, | ||
| # #275) the neural policy's buffer path requires. | ||
| ARG EIGS_REF=v0.32.0 | ||
| ARG EIGS_REF=v0.33.0 |
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.
Routine pin bump to the new release: v0.33.0, the silent-wrong-answer release.
Highlights that reach this repo: the plausible-but-wrong-at-rc=0 sweep (
regex_match#629,json_decode#628, four stdlib math bugs #638-#641, import top-level scope #673, the listcomp/local double binding #633/#642,report_valuecalling a converging sequence diverging #674), plus DEFLATE codecs (#684), tape-capturedclock_unix(#683), four C-backed list ops (#543/#544), andkill -USR1live observer dumps (#660).Verified before the bump, not after: this repo's suite was run locally against the v0.33.0 runtime and passed unchanged — no code change needed here.
That sweep was not a formality this time. It found real breakage in three siblings, fixed in their own PRs: EigenScript #630 widened
OP_LINE's operand from 16 to 32 bits, which is a bytecode ABI break for external producers (ouroboros produced empty output for every program, iLambdaAi's grader lost its top rung — both silently at rc=0), and #629 changedregex_match's shape for non-participating capture groups (EigenRegex's compat shim mirrored the old truncation deliberately).Release notes: https://github.com/InauguralSystems/EigenScript/releases/tag/v0.33.0
🤖 Generated with Claude Code