Skip to content
Permalink
Browse files
add missing cassert include -- should probably ultimately remove asse…
…rts instead
  • Loading branch information
jmalkin committed Apr 6, 2022
1 parent 1083233 commit 0e2883e06ddfab73e6a50cfc55743f1eff71bc02
Showing 2 changed files with 3 additions and 2 deletions.
@@ -75,8 +75,8 @@ py::list quantiles_sketch_get_quantiles(const quantiles_sketch<T>& sk,
bool inclusive) {
size_t n_quantiles = fractions.size();
auto result = inclusive
? sk.template get_quantiles<true>(&fractions[0], n_quantiles)
: sk.template get_quantiles<false>(&fractions[0], n_quantiles);
? sk.template get_quantiles<true>(&fractions[0], static_cast<uint32_t>(n_quantiles))
: sk.template get_quantiles<false>(&fractions[0], static_cast<uint32_t>(n_quantiles));

// returning as std::vector<> would copy values to a list anyway
py::list list(n_quantiles);
@@ -20,6 +20,7 @@
#ifndef _QUANTILES_SKETCH_IMPL_HPP_
#define _QUANTILES_SKETCH_IMPL_HPP_

#include <cassert>
#include <cmath>
#include <algorithm>
#include <stdexcept>

0 comments on commit 0e2883e

Please sign in to comment.