Skip to content

test_take #173

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 28, 2023
Merged

test_take #173

merged 2 commits into from
Mar 28, 2023

Conversation

honno
Copy link
Member

@honno honno commented Mar 15, 2023

Somewhat comprehensive test_take

@honno honno requested a review from asmeurer March 15, 2023 11:37
@asmeurer
Copy link
Member

Have you tried this on different libraries? I know pytorch is missing the axis argument, but numpy, cupy, numpy.array_api (git), and cupy.array_api (git) all have take that should be fully compliant.

@honno
Copy link
Member Author

honno commented Mar 16, 2023

Have you tried this on different libraries? I know pytorch is missing the axis argument, but numpy, cupy, numpy.array_api (git), and cupy.array_api (git) all have take that should be fully compliant.

I should of mentioned I really just developed this test using top-level numpy. Just tried this on cupy, array_api_compat.numpy and array_api_compat.cupy, which all pass this test in its current iteration. Will look at dev NumPy and CuPy later—didn't realise other folks been working on the .array_api namespaces, nice!

@asmeurer
Copy link
Member

Yeah I didn't either, though maybe we should try to nudge people to the compat library.

If this works with those libraries I'm +1 to merge.

Required for testing `torch_np`
@honno
Copy link
Member Author

honno commented Mar 28, 2023

The .array_api namespaces are kinda weird to test with as they don't implement complex dtypes, so I'll ignore 'em for now and get this merged given it works with NumPy-proper.

@honno honno merged commit a398866 into data-apis:master Mar 28, 2023
@honno honno deleted the test-take branch February 28, 2024 13:21
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