sim: consolidate to Tsimulation/{sim_v1,sim_v2}, retire v3 numbering - #546
Draft
ElmoPA wants to merge 1 commit into
Draft
sim: consolidate to Tsimulation/{sim_v1,sim_v2}, retire v3 numbering#546ElmoPA wants to merge 1 commit into
ElmoPA wants to merge 1 commit into
Conversation
Contributor
Author
This was referenced Aug 7, 2026
ElmoPA
force-pushed
the
sim/version-fork
branch
from
August 7, 2026 12:01
d80489a to
69d6836
Compare
ElmoPA
force-pushed
the
sim/converge-layout
branch
from
August 7, 2026 12:01
d7a9ee1 to
88d7366
Compare
ElmoPA
force-pushed
the
sim/version-fork
branch
from
August 7, 2026 12:11
69d6836 to
877c56e
Compare
ElmoPA
force-pushed
the
sim/converge-layout
branch
from
August 7, 2026 12:12
88d7366 to
35496aa
Compare
Replaces the three-version pushshapes/{v1,v2,v3} fork from the parent branch
with two standalone sibling packages, each carrying its own pushshapes/,
collect/, examples/ and tests/.
There is no v3. What was labelled v3 is the socket fix that ships AS v2 -- the
intermediate all-faces-grip build was a bug, never a release -- so SIM_VERSION
goes 3 -> 2. Content is otherwise identical to the old v3: obstacles, render
and shapes differ by 0 lines. sim_v1 is unchanged physics (imports and a stale
banner only), which is why every v1 dataset still replays bit-exact.
Tsimulation/__init__.py aliases the active version submodules to the top level,
so existing "from Tsimulation.pushshapes import X" call sites keep working.
Version selection moves to TSIM_VERSION (default sim_v2) -- per-process and
visible in the job environment, unlike the old symlink.
Verified by replay against real data: 74,001 episodes across 25 datasets. v1
sets reproduce at 100%, v2 sets at 92-99%. Post-consolidation regression:
probe_indomain_circle_v1 100%, u_socket_3000_v2 96%.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012V58H37tmcvgDthELMd5Xk
ElmoPA
force-pushed
the
sim/converge-layout
branch
from
August 7, 2026 12:23
35496aa to
5d46857
Compare
This was referenced Aug 21, 2026
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.

Replaces the three-version pushshapes/{v1,v2,v3} fork from the parent branch
with two standalone sibling packages, each carrying its own pushshapes/,
collect/, examples/ and tests/.
There is no v3. What was labelled v3 is the socket fix that ships AS v2 -- the
intermediate all-faces-grip build was a bug, never a release -- so SIM_VERSION
goes 3 -> 2. Content is otherwise identical to the old v3: obstacles, render
and shapes differ by 0 lines. sim_v1 is unchanged physics (imports and a stale
banner only), which is why every v1 dataset still replays bit-exact.
Tsimulation/init.py aliases the active version submodules to the top level,
so existing "from Tsimulation.pushshapes import X" call sites keep working.
Version selection moves to TSIM_VERSION (default sim_v2) -- per-process and
visible in the job environment, unlike the old symlink.
Verified by replay against real data: 74,001 episodes across 25 datasets. v1
sets reproduce at 100%, v2 sets at 92-99%. Post-consolidation regression:
probe_indomain_circle_v1 100%, u_socket_3000_v2 96%.
Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_012V58H37tmcvgDthELMd5Xk