Add permutation to random number generators
#3071
Comments
3 tasks
stress-tess
added a commit
to stress-tess/arkouda
that referenced
this issue
Apr 3, 2024
This PR (closes Bears-R-Us#3071) adds permutation to `rng`. The array version of the chpl `permute` proc requires idxType must be coercible from this stream’s eltType, so instead we use the domain version of `permute` and use that to gather the permuted vals. But we're not missing out on much because the array version seems to do the same thing but without aggregation
stress-tess
added a commit
to stress-tess/arkouda
that referenced
this issue
Apr 3, 2024
This PR (closes Bears-R-Us#3071) adds permutation to `rng`. The array version of the chpl `permute` proc requires idxType must be coercible from this stream’s eltType, so instead we use the domain version of `permute` and use that to gather the permuted vals. But we're not missing out on much because the array version seems to do the same thing but without aggregation
github-merge-queue bot
pushed a commit
that referenced
this issue
Apr 5, 2024
* Closes #3071: Add permutation to rng This PR (closes #3071) adds permutation to `rng`. The array version of the chpl `permute` proc requires idxType must be coercible from this stream’s eltType, so instead we use the domain version of `permute` and use that to gather the permuted vals. But we're not missing out on much because the array version seems to do the same thing but without aggregation * update in response to PR feedback and added float test --------- Co-authored-by: Tess Hayes <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add
permutationto our generators, we can get this easilypermutemethod defined on chpl generatorsRelevant numpy documentation:
The text was updated successfully, but these errors were encountered: