[incubator-datasketches-cpp] 07/07: Merge branch 'master' into tuple_sketch

[al...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

alsay pushed a commit to branch tuple_sketch
in repository https://gitbox.apache.org/repos/asf/incubator-datasketches-cpp.git

commit 67d7203595229704e7c0575dd9b1eef1f7e87b6e
Merge: 013b0d0 fa95167
Author: AlexanderSaydakov <Al...@users.noreply.github.com>
AuthorDate: Fri Aug 14 16:36:14 2020 -0700

    Merge branch 'master' into tuple_sketch

 .github/workflows/build_cmake.yml            |   2 +-
 .github/workflows/code_coverage.yml          |   2 +-
 .gitignore                                   |   1 -
 build/.gitignore                             |   7 ++
 common/include/serde.hpp                     |   9 +-
 kll/include/kll_quantile_calculator.hpp      |  37 ++++---
 kll/include/kll_quantile_calculator_impl.hpp | 159 ++++++++++++---------------
 kll/include/kll_sketch_impl.hpp              |  33 +++---
 python/src/cpc_wrapper.cpp                   |  41 ++++---
 python/src/fi_wrapper.cpp                    |  48 +++++---
 python/src/hll_wrapper.cpp                   | 101 ++++++++++-------
 python/src/kll_wrapper.cpp                   | 115 +++++++++++++++----
 python/src/theta_wrapper.cpp                 |  75 ++++++++-----
 python/src/vector_of_kll.cpp                 |  13 ++-
 python/src/vo_wrapper.cpp                    |  44 +++++---
 python/tests/kll_test.py                     |   8 +-
 python/tests/vector_of_kll_test.py           |  48 ++++++++
 theta/include/theta_intersection_impl.hpp    |   4 +-
 theta/include/theta_sketch_impl.hpp          |   2 +-
 theta/include/theta_union_impl.hpp           |   2 +-
 20 files changed, 481 insertions(+), 270 deletions(-)

diff --cc common/include/serde.hpp
index b8231c9,d0819d8..73e0901
--- a/common/include/serde.hpp
+++ b/common/include/serde.hpp
@@@ -47,10 -47,10 +47,10 @@@ template<typename T, typename Enable = 
  // with LongsSketch and ItemsSketch<Long> with ArrayOfLongsSerDe in Java
  template<typename T>
  struct serde<T, typename std::enable_if<std::is_arithmetic<T>::value>::type> {
 -  void serialize(std::ostream& os, const T* items, unsigned num) {
 +  void serialize(std::ostream& os, const T* items, unsigned num) const {
      bool failure = false;
      try {
-       os.write((char*)items, sizeof(T) * num);
+       os.write(reinterpret_cast<const char*>(items), sizeof(T) * num);
      } catch (std::ostream::failure& e) {
        failure = true;
      }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datasketches.apache.org
For additional commands, e-mail: commits-help@datasketches.apache.org