Skip to content

Commit

Permalink
Fix build and spelling errors for PR
Browse files Browse the repository at this point in the history
  • Loading branch information
ivberg committed May 2, 2024
1 parent 7fbe542 commit 2eec678
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion onnxruntime/core/common/logging/logging.cc
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ std::unique_ptr<ISink> EnhanceSinkWithEtw(std::unique_ptr<ISink> existingSink, l
// On non-Windows platforms, just return the existing logger
(void)originalSeverity;
(void)etwSeverity;
return existingLogger;
return existingSink;
#endif // _WIN32
}

Expand Down
14 changes: 7 additions & 7 deletions onnxruntime/core/platform/windows/telemetry.cc
Original file line number Diff line number Diff line change
Expand Up @@ -216,17 +216,17 @@ void WindowsTelemetry::LogSessionCreation(uint32_t session_id, int64_t ir_versio

// build the strings we need

std::string domain_to_verison_string;
std::string domain_to_version_string;
bool first = true;
for (auto& i : domain_to_version_map) {
if (first) {
first = false;
} else {
domain_to_verison_string += ',';
domain_to_version_string += ',';
}
domain_to_verison_string += i.first;
domain_to_verison_string += '=';
domain_to_verison_string += std::to_string(i.second);
domain_to_version_string += i.first;
domain_to_version_string += '=';
domain_to_version_string += std::to_string(i.second);
}

std::string model_metadata_string;
Expand Down Expand Up @@ -271,7 +271,7 @@ void WindowsTelemetry::LogSessionCreation(uint32_t session_id, int64_t ir_versio
TraceLoggingString(model_producer_version.c_str(), "modelProducerVersion"),
TraceLoggingString(model_domain.c_str(), "modelDomain"),
TraceLoggingBool(use_fp16, "usefp16"),
TraceLoggingString(domain_to_verison_string.c_str(), "domainToVersionMap"),
TraceLoggingString(domain_to_version_string.c_str(), "domainToVersionMap"),
TraceLoggingString(model_graph_name.c_str(), "modelGraphName"),
TraceLoggingString(model_metadata_string.c_str(), "modelMetaData"),
TraceLoggingString(loaded_from.c_str(), "loadedFrom"),
Expand All @@ -293,7 +293,7 @@ void WindowsTelemetry::LogSessionCreation(uint32_t session_id, int64_t ir_versio
TraceLoggingString(model_producer_version.c_str(), "modelProducerVersion"),
TraceLoggingString(model_domain.c_str(), "modelDomain"),
TraceLoggingBool(use_fp16, "usefp16"),
TraceLoggingString(domain_to_verison_string.c_str(), "domainToVersionMap"),
TraceLoggingString(domain_to_version_string.c_str(), "domainToVersionMap"),
TraceLoggingString(model_graph_name.c_str(), "modelGraphName"),
TraceLoggingString(model_metadata_string.c_str(), "modelMetaData"),
TraceLoggingString(loaded_from.c_str(), "loadedFrom"),
Expand Down

0 comments on commit 2eec678

Please sign in to comment.