ml-explore / ml-explore/mlx

Add full `array-api` tests

Open
#3,514 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement low priority python
Dominant language
C++
Stars
28.5k
Forks
2.3k
Avg merge
3d 8h
Merged PRs (30d)
62

Description

From our discussion here https://github.com/ml-explore/mlx/issues/48#issuecomment-4378140759 I agree with @mtsokol that the sensible thing to do would be not to one-off #3484 but instead to fix things wholesale by adding a full test suite from https://github.com/data-apis/array-api-tests

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Read the discussion linked from issue #3514 and inspect the existing work in #3484 first. Then review the data-apis/array-api-tests suite and determine how it should be integrated with MLX; done means the full suite is added and passes against MLX.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, python
Domain
testing
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.