Repository navigation
ci: optimize cognitive-shader-driver in member-tests (debug stays 0) - #1412
Merged
Merged
Conversation
member-tests hit its 30-minute limit on main (run 37766947128) and on #1410, both cut off in the hydrate step. The time went to one binary: the crossword_real_words_probe example tests (D-PUZZLE-0, added 2026-10-07) ran 810 s at opt-level 0. The shader-driver test step now passes --config 'profile.dev.package.cognitive-shader-driver.opt-level=3'. Only that package is optimized; debug info stays 0 from the manifest and every other crate keeps the shared opt-level-0 cache. Measured locally: 184 s for that binary, 4m17 for the whole step including the compile, all green. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EFw2WdKr1oxvaKCJC2ua2R
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing (1)
Warning Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption. Comment |
AdaWorldAPI
pushed a commit
that referenced
this pull request
Oct 8, 2026
member-tests hits its 30-minute limit on main too (runs 37766947128, 37806803926); #1412 is the fix. Identical change, so it no-ops once #1412 merges. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01R2EuNHrariL6B3G3agyASK
This was referenced Oct 8, 2026
AdaWorldAPI
marked this pull request as ready for review
October 8, 2026 17:37
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
AdaWorldAPI
pushed a commit
that referenced
this pull request
Oct 8, 2026
Conflicts only in the board: STATUS_BOARD's D-RPF-0 row takes main's test-pinned status (#1413), and the generated entries index is regenerated with entries_index.py. The plan's D-RPF-0 status note and the fold-join entry now cite #1413's pinned threshold half instead of calling it unexercised. The ported #1412 workflow change merged as a no-op. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EFw2WdKr1oxvaKCJC2ua2R
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.
member-testswas cancelled at its 30-minute limit, both onmain(run 37766947128) and on #1410. In both runs it was cut off in thehydratestep, after every earlier step had passed.Cause (measured from the job log)
The time does not come from compiling, and not from debug info:
RUSTFLAGS -C debuginfo=0and[profile.dev] debug = 0are already set. It comes from one test binary. The unit tests of thecrossword_real_words_probeexample (D-PUZZLE-0, added 2026-10-07) run a solver search that took 810 s at opt-level 0:every_question_counts_the_same_three_waysinstances_are_unique_and_givens_are_needednyt_lane_holds_one_claim_per_slot_of_its_own_gridFix
The shader-driver step now runs with:
Cargo.toml, so local dev builds are unaffected.#[ignore].Local measurement (
CARGO_PROFILE_DEV_DEBUG=0)crossword_real_words_probetestsThe "before" figure is from the CI run and the "after" figure is local, so they come from different machines.
🤖 Generated with Claude Code
https://claude.ai/code/session_01EFw2WdKr1oxvaKCJC2ua2R
Generated by Claude Code