apache / datasketches-cpp

Core C++ Sketch Library
https://datasketches.apache.org
Apache License 2.0
213 stars 70 forks source link

more generic implementation of array_of_doubles_sketch #399

Closed AlexanderSaydakov closed 8 months ago

AlexanderSaydakov commented 8 months ago

array_tuple_sketch , union, intersection, a_not_b with parameterized array type and policy

AlexanderSaydakov commented 8 months ago

Motivation: I found that array_ofdoubles was too rigid: doubles only, no way to change update policy and union policy, and no way to substitute our custom array with std::vector In this implementation all array_ofdoubles types are just aliases with default policy (if applicable) and array, but easy to have any type like std::vector or something. I am not sure if the name arraytuple* is the best. Suggestions are welcome.

coveralls commented 8 months ago

Pull Request Test Coverage Report for Build 6660511473


Totals Coverage Status
Change from base Build 6578009132: 0.0%
Covered Lines: 16070
Relevant Lines: 16242

💛 - Coveralls
jmalkin commented 8 months ago

Merge away!