Repository navigation
S05: licensed Phase A GUI acceptance - all nine cases PASS (record/evidence) - #245
Merged
Merged
Conversation
Real-ribbon runs on a fresh build of sow/2026-Q4 ad1f248 (S01-S04 merged) with the dependency stack rebuilt at its tips: Tas only, OpenStudio only, Both explicit (two sessions), Both embedded post-F1, multizone (two sessions), injected Tas refusal, cancel (Both, OpenStudio) and concurrent starts. Record, evidence and the S05 test harness; no product change. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
michaldengusiak
added a commit
that referenced
this pull request
Oct 20, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Roadmap S05 - Licensed Phase A GUI acceptance (SAM
documentation/SIMULATION_ROADMAP_Q4_2026.md). Record and evidence only: no product code changes. Record:documentation/S05-LicensedAcceptance.md. Evidence:documentation/evidence/s05-20261020/.Base
sow/2026-Q4ad1f2486, which has S01-S04 merged (S03 #2421e460bbe, S04 #243307cb0e3). The preflight (#24446a43d8d) was a rehearsal and is kept separate.Build under test
BuildAlls_v5 skip2027, 26 repositories, exit 0. The app was then built as Release into an isolated folder.APPDATAandUSERPROFILEwere redirected to a sandbox. Neither the installed SAM nor the real%APPDATA%\SAMwas touched.SAM\buildpredated SAM#212 (F1). All overnight real-engine evidence had therefore used a pre-F1SAM.Weather.dll. In this build,SAM.Weather,SAM.AnalyticalandSAM.Coreare byte-identical across the app, both producers andSAM\build.SAM_COMMIT. Per the owner decision of 2026-10-20 (temporary, until S06), it was set only for Both runs, to SAM62b54156. That is the clean commit the producers were verified to be built from. The manifests record it as a manual override.Matrix: all nine cases PASS (real ribbon, licensed TAS 9.5.7, OpenStudio 3.10 / E+ 25.1)
Succeeded, full provenance. Persisted results 602/602 exact vs the S01b baseline.Succeeded; CLI hash, E+ version fromeplusout.err. Snapshot without view state; result model with it.Comparedin both. Snapshot0c30db14…and canonical05c1e334…identical.comparison.csvidentical to S01b B2 and the preflight.embedded-corrected.epw; the preflight's matchedembedded-before.epw.File472ffb4e…in the manifest and in both producers.Compared; 2 spaces matched; identical snapshot and canonical hash. E+ values vary by about 1e-11 relative between sessions; this is known for this model.tas-run.jsonFailedwith the same text.Cancelled; no comparison and no result model; no engine process left.Test-harness defects (lost folder-dialog click, sandbox Desktop missing, BM_CLICK on an inactive box, a miscounted process wait, comparison-window refusal looked for at the wrong depth) are classified in the record. Each was fixed in the harness and the affected case rerun; attempts are kept as evidence. No product defect found.
Not exercised in the GUI: the Optimisation "Run Energy Simulation…" follow-up (the same guarded
SimulateTaspath, unit-tested in S02). Part O Hub is unguarded (S02 owner decision).Final PR head
4f0d12b0: hosted Build (39378742481) PASS, SPDX (39378742474) PASS.219f442bis the record and evidence;4f0d12b0adds the local unit-suite result on the rebuilt dependencies: 2926/2926.🤖 Generated with Claude Code