diff options
author | Arne Juul <arnej@verizonmedia.com> | 2020-09-28 14:39:49 +0000 |
---|---|---|
committer | Arne Juul <arnej@verizonmedia.com> | 2020-09-28 14:39:49 +0000 |
commit | 52e63a912e0dd705c2885f11e5f9230e3ccd8a22 (patch) | |
tree | 21d787dbd4b4335ffc04f15d9f1c3ccb5dc8f412 /eval | |
parent | ad6340c70b603b4e924088a8f396d92c03bcf944 (diff) |
use provided variables
Diffstat (limited to 'eval')
-rw-r--r-- | eval/src/vespa/eval/tensor/default_value_builder_factory.cpp | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/eval/src/vespa/eval/tensor/default_value_builder_factory.cpp b/eval/src/vespa/eval/tensor/default_value_builder_factory.cpp index d9167f3c9ee..74fd371e9a0 100644 --- a/eval/src/vespa/eval/tensor/default_value_builder_factory.cpp +++ b/eval/src/vespa/eval/tensor/default_value_builder_factory.cpp @@ -18,21 +18,21 @@ namespace { struct CreateDefaultValueBuilderBase { template <typename T> static std::unique_ptr<ValueBuilderBase> invoke(const ValueType &type, - size_t num_mapped_dims_in, - size_t subspace_size_in, + size_t num_mapped_dims, + size_t subspace_size, size_t expected_subspaces) { assert(check_cell_type<T>(type.cell_type())); if (type.is_double()) { return std::make_unique<DoubleValueBuilder>(); } - if (type.is_dense()) { - return std::make_unique<DenseTensorValueBuilder<T>>(type, subspace_size_in); + if (num_mapped_dims == 0) { + return std::make_unique<DenseTensorValueBuilder<T>>(type, subspace_size); } - if (type.is_sparse()) { - return std::make_unique<SparseTensorValueBuilder<T>>(type, num_mapped_dims_in, expected_subspaces); + if (subspace_size == 1) { + return std::make_unique<SparseTensorValueBuilder<T>>(type, num_mapped_dims, expected_subspaces); } - return std::make_unique<packed_mixed_tensor::PackedMixedTensorBuilder<T>>(type, num_mapped_dims_in, subspace_size_in, expected_subspaces); + return std::make_unique<packed_mixed_tensor::PackedMixedTensorBuilder<T>>(type, num_mapped_dims, subspace_size, expected_subspaces); } }; @@ -45,11 +45,11 @@ DefaultValueBuilderFactory DefaultValueBuilderFactory::_factory; std::unique_ptr<ValueBuilderBase> DefaultValueBuilderFactory::create_value_builder_base(const ValueType &type, - size_t num_mapped_dims_in, - size_t subspace_size_in, + size_t num_mapped_dims, + size_t subspace_size, size_t expected_subspaces) const { - return typify_invoke<1,TypifyCellType,CreateDefaultValueBuilderBase>(type.cell_type(), type, num_mapped_dims_in, subspace_size_in, expected_subspaces); + return typify_invoke<1,TypifyCellType,CreateDefaultValueBuilderBase>(type.cell_type(), type, num_mapped_dims, subspace_size, expected_subspaces); } //----------------------------------------------------------------------------- |