Skip to content

ci: optimize cognitive-shader-driver in member-tests (debug stays 0) - #1412

Merged
AdaWorldAPI merged 1 commit into
mainfrom
ccr-b2e415d9-4jfvyk-ci-optlevel
Oct 8, 2026
Merged

AdaWorldAPI merged 1 commit into
mainfrom
ccr-b2e415d9-4jfvyk-ci-optlevel

Conversation

@AdaWorldAPI

Copy link
Copy Markdown
Owner

member-tests was cancelled at its 30-minute limit, both on main (run 37766947128) and on #1410. In both runs it was cut off in the hydrate step, 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=0 and [profile.dev] debug = 0 are already set. It comes from one test binary. The unit tests of the crossword_real_words_probe example (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_ways
  • instances_are_unique_and_givens_are_needed
  • nyt_lane_holds_one_claim_per_slot_of_its_own_grid

Fix

The shader-driver step now runs with:

--config 'profile.dev.package.cognitive-shader-driver.opt-level=3'
  • Only this one package is compiled with optimization.
  • Debug info stays 0, from the manifest.
  • All other crates keep the shared opt-level-0 cache.
  • No change to Cargo.toml, so local dev builds are unaffected.
  • No test is skipped or marked #[ignore].

Local measurement (CARGO_PROFILE_DEV_DEBUG=0)

before after
crossword_real_words_probe tests 810 s (CI log) 184 s
whole shader-driver step ~15 min (CI) 4m17 including the compile, all green

The "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

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
@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 50819a8e-08fa-485b-a36a-32d5eb35209a
📥 Commits

Reviewing files that changed from the base of the PR and between 8fc2d08 and 7aeb258.

📒 Files selected for processing (1)
  • .github/workflows/rust-test.yml
 ___________________________________________________________________________________________________
< Test your software, or your users will. Test ruthlessly. Don't make your users find bugs for you. >
 ---------------------------------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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 @coderabbitai help to get the list of available commands.

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
@AdaWorldAPI
AdaWorldAPI marked this pull request as ready for review October 8, 2026 17:37
@AdaWorldAPI
AdaWorldAPI merged commit d0cb348 into main Oct 8, 2026
4 checks passed
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-08T17:38:20.666600Z 7aeb258 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants