From 51dd5fac79a16c0fc0d3f1302a1f41937ebbfef4 Mon Sep 17 00:00:00 2001 From: Daniel Han Date: Fri, 15 May 2026 10:37:37 -0700 Subject: [PATCH] ci: add tx >=5,<6 slow compile model_types to KNOWN_BROKEN_COMPILE (#5458) The per-model SIGALRM cap landed on the previous fix now exposes beit / sam / sam_hq as compile-too-slow on transformers >=5,<6 + trl >=1,<2 -- each exceeds the 60s per-model budget. They are real slow paths in unsloth_compile_transformers's source rewriter when handling beit / SAM's encoder layers on the new transformers line, not infra flakes (the prior fix logged sweep progress per 25 models so the slow ones are pinpointable in CI logs). Bucket them into Category F (compile exceeds budget) so the sweep stays green and each is tracked for follow-up zoo fixes in the same shape as the existing 27 known-broken entries. Surface behaviour stays identical: any NEW slow model_type still fails the cell with a TimeoutError tag. --- .github/workflows/consolidated-tests-ci.yml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/.github/workflows/consolidated-tests-ci.yml b/.github/workflows/consolidated-tests-ci.yml index 3f4e054b00..6b008d4bb1 100644 --- a/.github/workflows/consolidated-tests-ci.yml +++ b/.github/workflows/consolidated-tests-ci.yml @@ -962,6 +962,12 @@ jobs: # Category E: undefined name in emitted file. "perceiver": "name 'AbstractPreprocessor' is not defined", "sam3_lite_text": "name 'Sam3LiteTextLayerScaledResidual' is not defined", + # Category F: compile exceeds 60s budget on the runner. + # First seen on transformers >=5,<6; each represents a slow + # or recursive source-rewriter path the zoo can address. + "beit": "TimeoutError: compile exceeds per-model budget", + "sam": "TimeoutError: compile exceeds per-model budget", + "sam_hq": "TimeoutError: compile exceeds per-model budget", }