diff --git a/dorado/data_loader/ModelFinder.cpp b/dorado/data_loader/ModelFinder.cpp index f12045d3..1388108a 100644 --- a/dorado/data_loader/ModelFinder.cpp +++ b/dorado/data_loader/ModelFinder.cpp @@ -43,8 +43,8 @@ ModelSelection ModelComplexParser::parse(const std::string& arg) { spdlog::trace("Model option: '{}' unknown - assuming path", variant_str); selection.model = ModelVariantPair{model_variant}; } else { - spdlog::trace("'{}' found variant: '{}' and version: '{}'", variant_str), - to_string(model_variant), to_string(version); + spdlog::trace("'{}' found variant: '{}' and version: '{}'", variant_str, + to_string(model_variant), to_string(version)); selection.model = ModelVariantPair{model_variant, version}; } } else { diff --git a/dorado/models/models.cpp b/dorado/models/models.cpp index f2ccf3de..58489568 100644 --- a/dorado/models/models.cpp +++ b/dorado/models/models.cpp @@ -81,7 +81,7 @@ void suggest_models(const std::vector& models, const ModelVariantPair& model, const ModsVariantPair& mods) { if (Chemistry::NONE == chemistry) { - throw std::runtime_error("cannot get model without chemistry"); + throw std::runtime_error("Cannot get model without chemistry"); } if (mods.has_variant()) { @@ -154,18 +154,16 @@ ModelInfo find_model(const std::vector& models, const ModsVariantPair& mods, bool suggestions) { if (Chemistry::NONE == chemistry) { - throw std::runtime_error("cannot get model without chemistry"); + throw std::runtime_error("Cannot get model without chemistry"); } const auto matches = find_models(models, chemistry, model, mods); if (matches.empty()) { + spdlog::error("Failed to get {} model", description); if (suggestions) { - spdlog::error("Failed to get {} model. No matches for {}", description, - format_msg(chemistry, model, mods)); suggest_models(models, description, chemistry, model, mods); } - throw std::runtime_error("Failed to get " + description + " model. No matches for " + - format_msg(chemistry, model, mods)); + throw std::runtime_error("No matches for " + format_msg(chemistry, model, mods)); } // Get the only match or the latest model