Skip to content

Comments

More useful error for large indices#3079

Merged
awni merged 1 commit intomainfrom
large_index_conversion
Jan 29, 2026
Merged

More useful error for large indices#3079
awni merged 1 commit intomainfrom
large_index_conversion

Conversation

@awni
Copy link
Member

@awni awni commented Jan 29, 2026

Instead of bad_cast show a more useful error message for indexing arrays with > int32 indices.

Closes #3054

Copy link
Member

@angeloskath angeloskath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏

@awni awni merged commit fcbdd05 into main Jan 29, 2026
16 checks passed
@awni awni deleted the large_index_conversion branch January 29, 2026 21:02
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.

MLX Issues - 3 Model Failures, running through MlLX-VLM

2 participants