Skip to content
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

[WIP] Advanced Indexing #3: Added support for multiple multidimensional Indices #8912

Closed
wants to merge 10 commits into from

Conversation

kc611
Copy link
Contributor

@kc611 kc611 commented Apr 22, 2023

As titled this PR build over #8491 and adds support for more than one advanced index.

@sklam
Copy link
Member

sklam commented Apr 26, 2023

A test that i am trying. This one is from the first example in https://numpy.org/doc/stable/user/basics.indexing.html#combining-advanced-and-basic-indexing

from numba import njit
import numpy as np

@njit
def foo():
    x = np.arange(10 * 20 * 30).reshape(10, 20, 30)
    ind = np.arange(2 * 3 * 4).reshape(2, 3, 4)
    ind[ind >= 20] = 1
    return x[..., ind, :]

r = foo()
print(r)

@github-actions
Copy link

This pull request is marked as stale as it has had no activity in the past 3 months. Please respond to this comment if you're still interested in working on this. Many thanks!

@github-actions github-actions bot added the stale Marker label for stale issues. label Jul 26, 2023
@github-actions github-actions bot added the abandoned - stale PRs automatically closed due to stale status label Aug 3, 2023
@github-actions github-actions bot closed this Aug 3, 2023
@guilhermeleobas guilhermeleobas removed stale Marker label for stale issues. abandoned - stale PRs automatically closed due to stale status labels Nov 10, 2023
@guilhermeleobas guilhermeleobas added the abandoned PR is abandoned (no reason required) label Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 - In Progress abandoned PR is abandoned (no reason required)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants