Add logging and fix some error checking

src/c++/perf_analyzer/genai-pa/genai_pa/main.py

@@ -69,9 +69,13 @@ def report_output(metrics: LLMProfileData, args):
     if "concurrency_range" in args:
         infer_mode = "concurrency"
         load_level = args.concurrency_range
-    else:
+    elif "request_rate_range" in args:
         infer_mode = "request_rate"
         load_level = args.request_rate_range
+    else:
+        raise GenAiPAException(
+            "Neither concurrency_range nor request_rate_range was found in args when reporting metrics"
+        )
     # TODO: metrics reporter class that consumes Stats class for nicer formatting
     print(metrics.get_statistics(infer_mode, int(load_level)))
 

src/c++/perf_analyzer/genai-pa/genai_pa/parser.py

@@ -60,16 +60,6 @@ def _update_load_manager_args(args: argparse.ArgumentParser) -> argparse.Argumen
     return args
 
 
-def _verify_valid_arg_combination(
-    args: argparse.ArgumentParser,
-) -> argparse.ArgumentParser:
-    # Verify protocol and default url match
-    if getattr(args, "i") == "grpc" and getattr(args, "u") == DEFAULT_HTTP_URL:
-        setattr(args, "u", DEFAULT_GRPC_URL)
-
-    return args
-
-
 ### Handlers ###
 
 
@@ -231,11 +221,12 @@ def parse_args(argv=None):
     _add_dataset_args(parser)
 
     args, extra_args = parser.parse_known_args(argv)
-    # strip off the "--" demarking the pass through arguments
-    extra_args = extra_args[1:]
+    if extra_args:
+        # strip off the "--" demarking the pass through arguments
+        extra_args = extra_args[1:]
+        logger.info(f"Additional pass through args: {extra_args}")
 
     args = _update_load_manager_args(args)
-    args = _verify_valid_arg_combination(args)
     args = _prune_args(args)
 
     return args, extra_args