Improve error messaging

Author: atobiszei

src/graph_export/graph_export.cpp

@@ -96,7 +96,9 @@ static Status createTextGenerationGraphTemplate(const std::string& directoryPath
     SPDLOG_TRACE("modelsPath: {}, directoryPath: {}, ggufFilename: {}", modelsPath, directoryPath, ggufFilename.value_or("std::nullopt"));
     auto pluginConfigOrStatus = GraphExport::createPluginString(graphSettings.pluginConfig, exportSettings);
     if (std::holds_alternative<Status>(pluginConfigOrStatus)) {
-        return std::get<Status>(pluginConfigOrStatus);
+        auto status = std::get<Status>(pluginConfigOrStatus);
+        SPDLOG_ERROR("Failed to create plugin config: {}", status.string());
+        return status;
     }
     // clang-format off
     oss << R"(
@@ -429,7 +431,7 @@ std::variant<std::string, Status> GraphExport::createPluginString(const PluginCo
         name.SetString(pluginConfig.kvCachePrecision.value().c_str(), d.GetAllocator());
         auto itr = d.FindMember("KV_CACHE_PRECISION");
         if (itr != d.MemberEnd()) {
-            return StatusCode::PLUGIN_CONFIG_CONFLICTING_PARAMETERS;
+            return Status(StatusCode::PLUGIN_CONFIG_CONFLICTING_PARAMETERS, "Doubled KV_CACHE_PRECISION parameter in plugin config.");
         }
         d.AddMember("KV_CACHE_PRECISION", name, d.GetAllocator());
         configNotEmpty = true;
@@ -440,7 +442,7 @@ std::variant<std::string, Status> GraphExport::createPluginString(const PluginCo
         value.SetUint(pluginConfig.maxPromptLength.value());
         auto itr = d.FindMember("MAX_PROMPT_LEN");
         if (itr != d.MemberEnd()) {
-            return StatusCode::PLUGIN_CONFIG_CONFLICTING_PARAMETERS;
+            return Status(StatusCode::PLUGIN_CONFIG_CONFLICTING_PARAMETERS, "Doubled MAX_PROMPT_LEN parameter in plugin config.");
         }
         d.AddMember("MAX_PROMPT_LEN", value, d.GetAllocator());
         configNotEmpty = true;
@@ -451,7 +453,7 @@ std::variant<std::string, Status> GraphExport::createPluginString(const PluginCo
         value.SetString(pluginConfig.modelDistributionPolicy.value().c_str(), d.GetAllocator());
         auto itr = d.FindMember("MODEL_DISTRIBUTION_POLICY");
         if (itr != d.MemberEnd()) {
-            return StatusCode::PLUGIN_CONFIG_CONFLICTING_PARAMETERS;
+            return Status(StatusCode::PLUGIN_CONFIG_CONFLICTING_PARAMETERS, "Doubled MODEL_DISTRIBUTION_POLICY parameter in plugin config.");
         }
         d.AddMember("MODEL_DISTRIBUTION_POLICY", value, d.GetAllocator());
         configNotEmpty = true;
@@ -461,7 +463,7 @@ std::variant<std::string, Status> GraphExport::createPluginString(const PluginCo
         value.SetString(exportSettings.cacheDir.value().c_str(), d.GetAllocator());
         auto itr = d.FindMember("CACHE_DIR");
         if (itr != d.MemberEnd()) {
-            return StatusCode::PLUGIN_CONFIG_CONFLICTING_PARAMETERS;
+            return Status(StatusCode::PLUGIN_CONFIG_CONFLICTING_PARAMETERS, "Doubled CACHE_DIR parameter in plugin config.");
         }
         d.AddMember("CACHE_DIR", value, d.GetAllocator());
         configNotEmpty = true;