diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2023-09-04 11:40:48 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2023-09-04 11:40:48 +0000 |
commit | fd8d29f8e1ebb51decaccee71aa6119275c11be1 (patch) | |
tree | 9330d2ca829717fe27a53344ddc9a9e9bfc1f4bb /searchlib | |
parent | c3ee6199624f99eb695a0d397b800a0fd6ab326d (diff) |
Unify and modernize code and layout
Diffstat (limited to 'searchlib')
-rw-r--r-- | searchlib/src/vespa/searchlib/features/onnx_feature.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/searchlib/src/vespa/searchlib/features/onnx_feature.cpp b/searchlib/src/vespa/searchlib/features/onnx_feature.cpp index cdeb0515659..61731a1c09a 100644 --- a/searchlib/src/vespa/searchlib/features/onnx_feature.cpp +++ b/searchlib/src/vespa/searchlib/features/onnx_feature.cpp @@ -35,7 +35,8 @@ namespace search::features { namespace { -vespalib::string normalize_name(const vespalib::string &name, const char *context) { +vespalib::string +normalize_name(const vespalib::string &name, const char *context) { vespalib::string result; for (char c: name) { if (isalnum(c)) { @@ -50,7 +51,8 @@ vespalib::string normalize_name(const vespalib::string &name, const char *contex return result; } -vespalib::string my_dry_run(const Onnx &model, const Onnx::WireInfo &wire) { +vespalib::string +my_dry_run(const Onnx &model, const Onnx::WireInfo &wire) { vespalib::string error_msg; try { Onnx::EvalContext context(model, wire); @@ -132,8 +134,7 @@ OnnxBlueprint::setup(const IIndexEnvironment &env, return fail("model setup failed: %s", ex.what()); } Onnx::WirePlanner planner; - for (size_t i = 0; i < _model->inputs().size(); ++i) { - const auto &model_input = _model->inputs()[i]; + for (const auto & model_input : _model->inputs()) { auto input_feature = model_cfg->input_feature(model_input.name); if (!input_feature.has_value()) { input_feature = fmt("rankingExpression(\"%s\")", normalize_name(model_input.name, "input").c_str()); @@ -151,8 +152,7 @@ OnnxBlueprint::setup(const IIndexEnvironment &env, } } planner.prepare_output_types(*_model); - for (size_t i = 0; i < _model->outputs().size(); ++i) { - const auto &model_output = _model->outputs()[i]; + for (const auto & model_output : _model->outputs()) { auto output_name = model_cfg->output_name(model_output.name); if (!output_name.has_value()) { output_name = normalize_name(model_output.name, "output"); |