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

bug in reverse indexing of Strings #3821

Closed
ajpotts opened this issue Oct 8, 2024 · 0 comments · Fixed by #3830
Closed

bug in reverse indexing of Strings #3821

ajpotts opened this issue Oct 8, 2024 · 0 comments · Fixed by #3830
Assignees
Labels
bug Something isn't working

Comments

@ajpotts
Copy link
Contributor

ajpotts commented Oct 8, 2024

Describe the bug
The reverse indexing of Strings sometimes produces the incorrect result under reverse indexing on multi-local builds.

To Reproduce
With ./arkouda_server -nl 2 the code:

seed  = 8675309
size  = 100
s = ak.random_strings_uniform(1, 2, size, seed=seed)

s[::-1]

Gives output:


In [12]: 
    ...: s[::-1]
    ...: 
Out[12]: array(['A', 'A', 'A', 'A', 'A', 'A', 'A', 'C', 'C', 'C', 'C', 'C', 'C', 'V', 'V', 'V', 'V', 'V', 'V', 'U', 'U', 'U', 'U', 'U', 'U', 'T', 'T', 'T', 'T', 'T', 'T', 'T', 'C', 'C', 'C', 'C', 'C', 'C', 'C', 'C', 'C', 'C', 'C', 'C', 'L', 'L', 'L', 'L', 'L', 'L', 'X', 'X', 'X', 'X', 'X', 'X', 'X', 'D', 'D', 'D', 'D', 'D', 'D', 'B', 'B', 'B', 'B', 'B', 'B', 'A', 'A', 'A', 'A', 'A', 'A', 'Z', 'Z', 'Z', 'Z', 'Z', 'Z', 'Z', 'K', 'K', 'K', 'K', 'K', 'K', 'N', 'N', 'N', 'N', 'N', 'N', 'E', 'E', 'E', 'E', 'E', 'E'])

In [14]: s
Out[14]: array(['E', 'W', 'V', 'S', 'K', 'Z', 'N', 'W', 'C', 'Z', 'B', 'F', 'K', 'T', 'G', 'D', 'E', 'Q', 'Z', 'N', 'K', 'Z', 'O', 'X', 'S', 'A', 'Q', 'Z', 'M', 'H', 'D', 'B', 'H', 'X', 'D', 'P', 'C', 'D', 'N', 'D', 'P', 'C', 'Q', 'X', 'X', 'C', 'R', 'W', 'H', 'G', 'L', 'A', 'U', 'A', 'V', 'V', 'C', 'F', 'X', 'I', 'R', 'A', 'C', 'V', 'B', 'B', 'H', 'K', 'T', 'Z', 'N', 'O', 'J', 'I', 'G', 'U', 'K', 'T', 'X', 'X', 'L', 'V', 'W', 'J', 'W', 'S', 'G', 'C', 'U', 'Y', 'M', 'O', 'A', 'A', 'W', 'O', 'E', 'Q', 'K', 'G'])

Expected behavior
The array should be correctly reversed.

@ajpotts ajpotts added the bug Something isn't working label Oct 8, 2024
stress-tess added a commit to stress-tess/arkouda that referenced this issue Oct 9, 2024
…ocale

This PR fixes Bears-R-Us#3821 which gave incorrect results for stridable index into segstrngs on multilocale
github-merge-queue bot pushed a commit that referenced this issue Oct 10, 2024
* Fixes #3821: Bug in stridable indexing of Strings in multilocale

This PR fixes #3821 which gave incorrect results for stridable index into segstrngs on multilocale

* remove test skip

---------

Co-authored-by: Tess Hayes <[email protected]>
Co-authored-by: ajpotts <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants