Re-pin to ost 0.22.8, and close PKG-4 and PKG-5 on the measurement - #151
Merged
Conversation
0.22.8 is current, so the contract takes it. What it does not do is close anything, and that is recorded where the next reader will look rather than left for them to assume. Re-pulling a pinned leaf under 0.22.8 leaves `pxrConfig.cmake` and `cmake/pxrTargets.cmake` with the same MD5s they had under 0.22.6, producer's Python include directory and all -- report 37's P1 is in what was exported, not in the tool that pulls it, so only a republished runtime can move it. And `ost ci generate` renders the same lanes: the whole diff is the version string, in the bootstrap step, the version assertion and the cache key. Measured on the way, and worth having on record: 0.22.8's `runtime validate` passes every check on this workstation including `openusd-render`, which 0.22.6 skipped here. One of those checks is `consumer-configure` -- *find_package(pxr) configures against <runtime>* -- and it passed on the Linux runner too, on the same runtime the consumer lane could not link against. It is presumably configure-only; either way it is not a substitute for that lane. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
PKG-4 and PKG-5 close by measurement. The lane is green on `windows-2022`, `macos-15` and `ubuntu-24.04`, twelve packages each, and criterion 6 is met: every workspace target in every closure is present on all three platforms or on none, and the one difference PACKAGE_CONTRACT.md permits is present in both directions -- `Threads::Threads` on macOS and Linux, `ws2_32` on Windows, for `liveTransport` and each of the three adapters that inherit it. That closes the raw-library half of #113, which had been open since the mocopi receiver grew a platform link: on both POSIX platforms `vrmAdapterMocopi` links the threading library and **no** socket one, which is the absence no Windows run can see. The carried entry leaves the roadmap, because the roadmap holds work that is not finished. Every `Standalone` cell in section 4 is now unqualified. The four that read **measured (Windows)** were `liveTransport` and the three adapters; nothing about their packages changed, and what changed is that a second and third host looked. Two things the run said that the plan had not: The repair fires on two platforms and not on macOS, whose baked paths resolve there -- so **neither workaround has ever been exercised on macOS**, and the lane prints that on every run rather than implying coverage it does not have. And the synthetic verification of the criterion-6 comparison had the right shape and the wrong counts: OpenUSD's platform link line is three entries on Windows, two on Linux and three on macOS, where the prediction had three and three. The rules fired where they were aimed; what they would see was approximate, which is the limit that was stated in advance rather than after. Co-Authored-By: Claude Opus 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.
Two things, in that order: the
ostpin moves to 0.22.8, and PKG-4/PKG-5 closeon the measurement that #150
could not carry because its lane had not run yet.
Closes #113.
The re-pin changes nothing here, and that is the finding
0.22.8 is current, so the contract takes it. What it does not do is close
anything, and that is on record rather than left to be assumed:
MD5s for
pxrConfig.cmakeandcmake/pxrTargets.cmakeas under 0.22.6, andthe producer's Python include directory is still on sixteen imported targets.
Report 37's
P1 is in what was exported, so only a republished runtime can move it.
ost ci generaterenders no semantic change. The whole regeneration diffis the version string, in three places per job.
runtime validateis fully green on a workstation under 0.22.8,openusd-renderincluded, which 0.22.6 skipped there.One measurement from that worth reading:
runtime validatehas aconsumer-configurecheck — find_package(pxr) configures against<runtime> — and it passed on the Linux runner, on the same runtime the
consumer lane could not link against. It is presumably configure-only, since a
probe that linked an imported target would have hit the same wall. It is not a
substitute for this lane, and reading it as one is the same mistake as reading
the workspace cell's green as evidence.
PKG-4 and PKG-5 close by measurement
The lane is green on
windows-2022,macos-15andubuntu-24.04— twelvepackages each — and criterion 6 is met:
on none;
directions —
Threads::Threadson macOS and Linux,ws2_32on Windows, forliveTransportand each of the three adapters that inherit it.That second line is #113's raw-library half, open since the mocopi receiver grew
a platform link: on both POSIX platforms
vrmAdapterMocopilinks the threadinglibrary and no socket one, which is the absence no Windows run can see. The
carried roadmap entry goes with it, because the roadmap holds work that is not
finished.
Every
Standalonecell in PACKAGE_CONTRACT.md §4 is now unqualified. The fourthat read measured (Windows) were
liveTransportand the three adapters;nothing about their packages changed — a second and third host looked.
Two things the run said that the plan had not
The repair fires on two platforms, not three. macOS's baked paths resolve on
macos-15, so it repaired nothing and derived nothing — neither workaroundhas ever been exercised there. That is a latent gap, not a passing grade: if
that leaf is republished from a machine whose paths do not resolve on a runner,
macOS fails with nothing to fall back on. It is left measured-and-stated rather
than fixed, because fixing a
sysconfigderivation for a host this projectcannot run is guesswork — and the lane prints the gap on every run.
The synthetic verification had the right shape and the wrong counts. The
criterion-6 comparison's ten cases were checked against predicted POSIX
closures, and #150 said in advance that this was a prediction. Measured:
OpenUSD's platform link line is three entries on Windows (
Dbghelp.lib,Shlwapi.lib,Ws2_32.lib), two on Linux (dl,m) and three on macOS(those two plus
-framework Foundation) — 10 external entries against 9 and 9,where the prediction had 10 against 10. The rules fired where they were aimed;
what they would see was approximate.
What is still not known
Why
workspace-pr-linuxis green on a runner where this lane measures/usr/include/python3.13missing. Three explanations were tested andeliminated — the toolchain file, the Ninja generator, and a repair inside
ost build(which fails against a foreign path exactly as a plaincmakedoes). Report 37 records it as unexplained rather than guessing, and the lane
now prints the underlying fact on every run.
🤖 Generated with Claude Code