diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2023-09-04 17:00:01 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-09-04 17:00:01 +0200 |
commit | 1deac7adfaa6583a19cfe564de07deca2a0835b0 (patch) | |
tree | 5877cccf6f0459e5f2cb5eb03cd01c96f46a4bca /searchlib | |
parent | 03d68b13d0cf62446631bb7856307892200e43af (diff) | |
parent | 601af030f40b9b7d270e59fbff68dbf32583fef4 (diff) |
Merge pull request #28376 from vespa-engine/balder/minor-cleanup
Unify and modernize code and layout
Diffstat (limited to 'searchlib')
-rw-r--r-- | searchlib/src/vespa/searchlib/features/onnx_feature.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/searchlib/src/vespa/searchlib/features/onnx_feature.cpp b/searchlib/src/vespa/searchlib/features/onnx_feature.cpp index cdeb0515659..a330a4ff325 100644 --- a/searchlib/src/vespa/searchlib/features/onnx_feature.cpp +++ b/searchlib/src/vespa/searchlib/features/onnx_feature.cpp @@ -132,8 +132,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 +150,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"); |