Remove loadgen warnings

mlcommons · Feb 5, 2024 · 86bfe86 · 86bfe86
1 parent 473053f
commit 86bfe86
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/loadgen/results.cc b/loadgen/results.cc
@@ -431,6 +431,8 @@ void PerformanceSummary::LogSummary(AsyncSummary& summary) {
         summary("Tokens per second: ", tokens_per_second);
         break;
       }
+      case TestScenario::Server:
+        break;
     }
   }
 

diff --git a/loadgen/test_settings_internal.cc b/loadgen/test_settings_internal.cc
@@ -690,12 +690,13 @@ int TestSettings::FromConfig(const std::string &path, const std::string &model,
   lookupkv(model, scenario, "test05_schedule_rng_seed", &test05_schedule_rng_seed, nullptr);
 
   // keys that apply to token metrics
-  if (lookupkv(model, scenario, "use_token_latencies", &val, nullptr))
+  if (lookupkv(model, scenario, "use_token_latencies", &val, nullptr)){
     use_token_latencies = (val == 1) ? true : false;
     if (use_token_latencies){
       lookupkv(model, "Server", "ttft_latency", &server_ttft_latency, nullptr, 1000 * 1000);
       lookupkv(model, "Server", "tpot_latency", &server_tpot_latency, nullptr, 1000 * 1000);
     }
+  }
 
   // keys that apply to SingleStream
   lookupkv(model, "SingleStream", "target_latency_percentile", nullptr,

diff --git a/loadgen/test_settings_internal.h b/loadgen/test_settings_internal.h
@@ -83,8 +83,8 @@ struct TestSettingsInternal {
 
   bool sample_concatenate_permutation;
   bool use_token_latencies = false;
-  uint64_t server_ttft_latency;
-  uint64_t server_tpot_latency;
+  int64_t server_ttft_latency;
+  int64_t server_tpot_latency;
 };
 
 /// \brief A namespace of collections of FindPeakPerformance helper functions,