chore(deps): sync all upstream submodules to resolve Cmlx build error - #43
Merged
Conversation
solderzzc
force-pushed
the
fix/dependency-sync-final
branch
from
April 16, 2026 04:43
e3b8b11 to
06eb018
Compare
solderzzc
added a commit
that referenced
this pull request
Aug 6, 2026
Re-points the bump at 717d77f9, which now carries both upstream fixes: - #43: non-indexed .safetensors ignored + RMSNorm shift guard (fixes #118) - #44: Gemma 4 KV-shared layers + quantizable scaled projections (fixes #120) With #44 included, the audio and omni integration jobs pass locally; opencode reaches its npm-install step (already continue-on-error). dflash remains red until #121 is fixed. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
solderzzc
added a commit
that referenced
this pull request
Aug 6, 2026
* chore: bump mlx-swift-lm for the non-indexed weights fix (#118) Picks up SharpAI/mlx-swift-lm#43 (e60ccfd), which stops loadWeights from sweeping .safetensors that the weight index does not list, and guards the Qwen35 RMSNorm shift on the weights actually looking zero-centered. Fixes #118. mlx-community/Qwen3.6-27B-OptiQ-4bit previously loaded without error and generated noise, because its out-of-index optiq/mtp.safetensors made sanitize add 1 to every norm weight. Verified against that model on this commit: the text path answers correctly with MTP disabled and enabled, and the vision path no longer fails with `Unhandled keys ["mtp"]`. No change for Qwen3.5-4B-4bit or Qwen3.6-35B-A3B-4bit. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * chore: advance mlx-swift-lm to include the Gemma 4 KV-shared fix Re-points the bump at 717d77f9, which now carries both upstream fixes: - #43: non-indexed .safetensors ignored + RMSNorm shift guard (fixes #118) - #44: Gemma 4 KV-shared layers + quantizable scaled projections (fixes #120) With #44 included, the audio and omni integration jobs pass locally; opencode reaches its npm-install step (already continue-on-error). dflash remains red until #121 is fixed. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
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
1 participant
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.
This mega-bump pulls in the mlx-swift Cmlx bug fix and the Swift 6 compiler warnings fix from mlx-swift-lm PR #18. Please merge this to stabilize the main branch so you can safely close PRs 38, 39, 40, and 41.