Address Copilot round 4: add PrintUsage on missing values, use ToUpperInvariant

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: Copilot

Samples/WindowsML/Shared/cpp/ArgumentParser.cpp

@@ -41,6 +41,7 @@ namespace Shared
                 if (i + 1 >= arguments.size())
                 {
                     std::wcout << L"ERROR: --model requires a value.\n";
+                    PrintUsage();
                     return false;
                 }
                 options.model_path = arguments[++i];
@@ -50,6 +51,7 @@ namespace Shared
                 if (i + 1 >= arguments.size())
                 {
                     std::wcout << L"ERROR: --compiled_output requires a value.\n";
+                    PrintUsage();
                     return false;
                 }
                 options.output_path = arguments[++i];
@@ -59,6 +61,7 @@ namespace Shared
                 if (i + 1 >= arguments.size())
                 {
                     std::wcout << L"ERROR: --image_path requires a value.\n";
+                    PrintUsage();
                     return false;
                 }
                 options.image_path = arguments[++i];
@@ -68,6 +71,7 @@ namespace Shared
                 if (i + 1 >= arguments.size())
                 {
                     std::wcout << L"ERROR: --ep_policy requires a value.\n";
+                    PrintUsage();
                     return false;
                 }
                 auto policy_str = arguments[++i];
@@ -108,6 +112,7 @@ namespace Shared
                 if (i + 1 >= arguments.size())
                 {
                     std::wcout << L"ERROR: --perf_mode requires a value.\n";
+                    PrintUsage();
                     return false;
                 }
                 std::wstring perf_mode_token = std::wstring(arguments[++i]);
@@ -135,6 +140,7 @@ namespace Shared
                 if (i + 1 >= arguments.size())
                 {
                     std::wcout << L"ERROR: --ep_name requires a value.\n";
+                    PrintUsage();
                     return false;
                 }
                 options.ep_name = arguments[++i];
@@ -144,6 +150,7 @@ namespace Shared
                 if (i + 1 >= arguments.size())
                 {
                     std::wcout << L"ERROR: --device_type requires a value.\n";
+                    PrintUsage();
                     return false;
                 }
                 std::wstring dt = std::wstring(arguments[++i]);

Samples/WindowsML/Shared/cs/ArgumentParser.cs

@@ -62,7 +62,7 @@ public static Options ParseOptions(string[] args)
                         if (i + 1 < args.Length)
                         {
                             string policyStr = args[++i];
-                            switch (policyStr.ToUpper())
+                            switch (policyStr.ToUpperInvariant())
                             {
                                 case "NPU":
                                     options.EpPolicy = ExecutionProviderDevicePolicy.PREFER_NPU;