Repository navigation
Conversation
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
|
/ok to test aaf85bf |
Diff stats comparison, PR 3182 stack vs this PRBoth diffs use the same
Code/config excludes tests and Markdown, reStructuredText, and Mermaid documentation. Its added lines decreased 3.7%; the existing-file footprint remains 26 of 101 files in both versions. — codex GPT-6 on behalf of Ralf — |
|
Narration sketch for PR 3309This outline captures the narration approach considered alongside splitting the This sketch refers to PR 3309 at aaf85bf, How each stop would workStart with the intended behavior, trace one example, inspect one or two relevant 1 Establish the starting pointMain already contains both bindings source roots. Its existing pipeline builds Draw that before-and-after picture. Both roots produce the same Checkpoint: trace which source tree and which wheel supply bindings to a CUDA 12 Open: the existing prior-branch download 2 Explain package and version selectionFollow three builds: the default CUDA 13 metapackage, an explicitly selected Then explain the maintenance-version bootstrap. At the baseline, Identify three product choices separately: the CUDA 12 Python 3.11 minimum, Checkpoint: predict the version and dependency family of each example before Open: metapackage selection 3 Follow one CUDA 12 source change through the plannerUse a handwritten CUDA 12 bindings edit as the example. Show the resulting build Compare this with a Core-only edit and a shared-helper edit. Explain why builds Checkpoint: which decisions are required for correctness, and which save CI Open: the planner. 4 Follow wheels into actual testsDraw the producer-consumer graph: Pathfinder, each bindings line, both Core ABI Compare a local-bindings GPU row with a published-floor row. Explain the purpose A required local build or artifact failure must stay a failure. Any temporary Checkpoint: if local CUDA 12 wheels appeared in a new job tomorrow, which Open: wheel orchestration 5 Trace a release from its tagUse a hypothetical Make artifact compatibility explicit. Changing the metapackage artifact name or Checkpoint: after each proposed merge, could we still release CUDA 13 while Open: tagged-source resolution 6 Review the small behavior and policy changesCompare each of the three runtime backports with its CUDA 13 counterpart: byte Discuss the maintenance policy and manual-backport workflow separately. Decide Checkpoint: which of these changes can be understood, tested, and merged before 7 Challenge proportionality and choose a landing strategyFor the checks and tests we encountered, ask what wrong outcome could otherwise Use the recorded manual validation run Finish by agreeing on the next functional landing state and its acceptance test. — codex GPT-6 on behalf of Ralf — |
Alternative route: seven independently mergeable PRsAn alternative direction is a staged migration through seven individually reviewable and mergeable PRs, using #3309 as the complete reference implementation. Each merged PR should leave a working state that can stand on its own while later stages are unfinished. Main already combines local CUDA 13 builds with CUDA 12 bindings from
Release continuity matters alongside green test CI. The existing downloader uses broad component patterns. Introducing another major's artifacts can affect existing releases immediately. Every stage must preserve those contracts through explicit selection or separate artifact names. The runtime backports, Python minimum, wheel payload, and metapackage dependency policy need explicit placement when refining these boundaries. A retained version milestone also needs deliberate SCM and tag-CI preparation. This approach costs temporary compatibility, some duplicate CI work, and validation of each intermediate state. Total migration churn may increase, and central build/test increments are likely to remain several hundred lines. These are proposed boundaries; the intermediate states still need implementation and validation. The first substantial test of the decomposition is a working local CUDA 12 build/test path that leaves existing CUDA 13 CI and releases usable. Use that increment, together with the narration, to refine the series before undertaking the full migration. — codex GPT-6 on behalf of Ralf — |
Signed-off-by: Ralf W. Grosse-Kunstleve <[email protected]>
Signed-off-by: Ralf W. Grosse-Kunstleve <[email protected]>
Signed-off-by: Ralf W. Grosse-Kunstleve <[email protected]>
Signed-off-by: Ralf W. Grosse-Kunstleve <[email protected]>
c0546cc to
4acf836
Compare
— codex GPT-6 on behalf of Ralf —
Work in progress: awaiting CI and an interactive walkthrough.
Consolidates the six-PR stack ending at #3182 onto current
main: shared builds, tests, documentation, and releases for CUDA 12/13 bindings. Trims redundant CI plumbing and implementation-shaped tests while retaining release and artifact checks.Local validation: 347 CI-tool tests and full pre-commit.