Skip to content

fix: error: no such module 'Tokenizers' #143

Closed
christophsturm wants to merge 3 commits into
Blaizzy:mainfrom
christophsturm:fix/consumer-graph-tokenizers
Closed

fix: error: no such module 'Tokenizers' #143
christophsturm wants to merge 3 commits into
Blaizzy:mainfrom
christophsturm:fix/consumer-graph-tokenizers

Conversation

@christophsturm

Copy link
Copy Markdown

No description provided.

@lucasnewman

Copy link
Copy Markdown
Collaborator

@christophsturm Do you have a test case where this isn't working? This doesn't seem right to me -- it should just work when pulling in the dependency as a package.

@christophsturm

christophsturm commented Apr 6, 2026

Copy link
Copy Markdown
Author

@lucasnewman my pr contains a test project so if you comment out the fix that should fail.

you can just change the dependency of the test project to
.package(
url: "https://github.com/Blaizzy/mlx-audio-swift.git",
revision: "2fd41458059e2d80403436167d5263f585d120d4"
),
to see the error.

@lucasnewman

Copy link
Copy Markdown
Collaborator

Ok, thanks. It took me a while to hunt this down -- it looks like there was a change in mlx-swift-lm in ml-explore/mlx-swift-lm#118 which triggers this. We're pinned to major version 2 so it shouldn't immediately impact anything unless you've modified the package locally.

@christophsturm

Copy link
Copy Markdown
Author

right so my fix improves the situation when using this library together with other versions.

@lucasnewman

Copy link
Copy Markdown
Collaborator

Resolved in #158

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