Repository navigation
Modernize CI platforms and repair source builds and malformed-input handling - #125
Open
afedynitch wants to merge 1 commit into
Open
afedynitch wants to merge 1 commit into
afedynitch wants to merge 1 commit into
Conversation
|
coverage: 96.838% (+0.006%) from 96.832% — ci/modernize-platforms into main |
…andling Retire Python 3.9 and Intel macOS, follow maintained runner and wheel-image defaults, and reduce version pins. Include bundled compression headers and license in source archives. Advance HepMC3 to its focused ASCII-reader fixes and cover truncated inputs with and without a final newline. Bound tests and expose native deadlock diagnostics. Assisted-by: Codex:GPT-6
afedynitch
force-pushed
the
ci/modernize-platforms
branch
from
October 29, 2026 16:21
b00ba46 to
aaa0a30
Compare
This branch has not been deployed
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.
CI source builds omitted HepMC3 compression headers, and malformed ASCII input could stall Windows debug tests. This PR repairs those failures and refreshes the supported platform matrix.
#124 is merged. This PR is rebased onto
mainand contains one modernization commit.Graphviz setup on macOS uses the runner's Homebrew snapshot, skipping explicit/automatic updates and install cleanup. The old macOS 14 job compiled dependencies from source (CMake alone took ten minutes); the current runner uses prebuilt bottles. Avoiding the redundant update also avoids unrelated toolchain migrations.
Validation: the previous head b00ba46 passed all hosted tests, docs, coverage, hooks, source-package and platform wheel builds. The only tracked change since that green tree is six lines configuring macOS Graphviz setup. Local hooks pass; hosted validation of the new head is running.
Windows investigation was limited to two existing-run analyses. Native compilation took about 101–104 seconds; the first PNG-rendering test took 4.0–4.9 seconds versus about 0.24 seconds for the next equivalent test. This suggests a cold-start effect but does not establish filesystem caching as the cause. Windows behavior is unchanged; deeper profiling is deferred to local Windows investigation.
AI assistance: implementation, investigation, restructuring and this description were prepared with Codex (GPT-6).