arkouda icon indicating copy to clipboard operation
arkouda copied to clipboard

refactor `RadixSortLSD` so that multiple sort configurations can be instantiated at the same time

Open mhmerrill opened this issue 3 years ago • 1 comments

refactor RadixSortLSD so that multiple sort configurations can be instantiated at the same time. This will enable us to to support multiple sort configurations at the same time.

mhmerrill avatar Apr 28 '22 17:04 mhmerrill

from our discussions I think we are going to:

  • add a new Msg function to the server to generate RadixSortLSDPlans calling makeRadixSortLSDPlan() and caching the plans in a map.
  • have all places which use the sort call chooseRadixSortLSDPlan() with the primary argument to this function being the input array size.

This will allow us to encapsulate all the parameters which are used to make a plan into one place in the code instead of reusing them in 20+ source modules.

mhmerrill avatar Jul 12 '22 15:07 mhmerrill

This work did not yield the benefits initially expected. Additionally, there were some blocking complications. This is no longer being pursued.

Ethan-DeBandi99 avatar Nov 29 '22 17:11 Ethan-DeBandi99