Skip to content

Commit

Permalink
add annotation that impacted algorithm used in Akrouda sort tests (#2…
Browse files Browse the repository at this point in the history
…5993)

This PR adds an annotation to a couple of the Arkouda graphs.

(noted by @jeremiah-corrado): For the sorting tests, the historical
behavior was buggy (i.e., for real-valued arrays we were completely
ignoring the algorithm argument and always used the LSD algorithm).
After Bears-R-Us/arkouda#3602 (merged on 8/7),
we are now using the algorithm argument, so the LSD and MSD lines have
different performance.

[reviewed-by: nobody; annotations update]
  • Loading branch information
stonea authored Sep 25, 2024
2 parents 1a61cd3 + 91861a7 commit a5e0c9c
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions test/ANNOTATIONS.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2537,6 +2537,11 @@ arkouda-array_create:
04/05/24:
- Closes 3071 Add permutation to our generators (Bears-R-Us/arkouda#3078)

arkouda-sort-cases:
<<: *arkouda-base
08/07/24:
- Closes 3526 refactor argsortMsg to remove registerND annotation (Bears-R-Us/#3602)

shoc-triad-all-lg:
11/10/22:
- Ensure we time mem transfer in SHOC Triad (#20961)
Expand Down

0 comments on commit a5e0c9c

Please sign in to comment.