From f10d7931d8017592d7da462568ffa175b4d7fe81 Mon Sep 17 00:00:00 2001 From: Arjun Suresh Date: Wed, 19 Feb 2025 22:02:02 +0000 Subject: [PATCH 1/2] Update check.py | Fix for the compliance checker for multi analyzer config --- compliance/check.py | 48 ++++++++++++++++++++++----------------------- 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/compliance/check.py b/compliance/check.py index 06b058d..1a8d379 100755 --- a/compliance/check.py +++ b/compliance/check.py @@ -706,34 +706,34 @@ def check_ptd_config(server_sd: SessionDescriptor) -> None: """ ptd_config = server_sd.json_object["ptd_config"] - dev_num = ptd_config["device_type"] - assert ( - dev_num in SUPPORTED_MODEL.values() - ), f"Device number {dev_num} is not supported. Supported numbers are " + ", ".join( - [str(i) for i in set(SUPPORTED_MODEL.values())] - ) + for analyzer in ptd_config: + dev_num = analyzer["device_type"] + assert ( + dev_num in SUPPORTED_MODEL.values() + ), f"Device number {dev_num} is not supported. Supported numbers are " + ", ".join( + [str(i) for i in set(SUPPORTED_MODEL.values())] + ) - if dev_num == 77: - channels = "" - command = ptd_config["command"] + if dev_num == 77: + channels = "" + command = analyzer["command"] - for i in range(len(command)): - if command[i] == "-c": - channels = command[i + 1] - break + for i in range(len(command)): + if command[i] == "-c": + channels = command[i + 1] + break - dev_name = "" - for name, num in SUPPORTED_MODEL.items(): - if num == dev_num: - dev_name = name - break - - assert ( - len(channels.split(",")) == 2 - and ptd_config["channel"] - and len(ptd_config["channel"]) == 2 - ), f"Expected multichannel mode for {dev_name}, but got 1-channel." + dev_name = "" + for name, num in SUPPORTED_MODEL.items(): + if num == dev_num: + dev_name = name + break + assert ( + len(channels.split(",")) == 2 + and analyzer["channel"] + and len(analyzer["channel"]) == 2 + ), f"Expected multichannel mode for {dev_name}, but got 1-channel." def debug_check(server_sd: SessionDescriptor) -> None: """Check debug is disabled on server-side""" From 5f7767b4e94ca440d1fa0965a20b48653e3ccb4a Mon Sep 17 00:00:00 2001 From: mlcommons-bot Date: Wed, 19 Feb 2025 22:02:16 +0000 Subject: [PATCH 2/2] [Automated Commit] Format Codebase --- compliance/check.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/compliance/check.py b/compliance/check.py index 1a8d379..a34a45c 100755 --- a/compliance/check.py +++ b/compliance/check.py @@ -708,10 +708,9 @@ def check_ptd_config(server_sd: SessionDescriptor) -> None: for analyzer in ptd_config: dev_num = analyzer["device_type"] - assert ( - dev_num in SUPPORTED_MODEL.values() - ), f"Device number {dev_num} is not supported. Supported numbers are " + ", ".join( - [str(i) for i in set(SUPPORTED_MODEL.values())] + assert dev_num in SUPPORTED_MODEL.values(), ( + f"Device number {dev_num} is not supported. Supported numbers are " + + ", ".join([str(i) for i in set(SUPPORTED_MODEL.values())]) ) if dev_num == 77: @@ -735,6 +734,7 @@ def check_ptd_config(server_sd: SessionDescriptor) -> None: and len(analyzer["channel"]) == 2 ), f"Expected multichannel mode for {dev_name}, but got 1-channel." + def debug_check(server_sd: SessionDescriptor) -> None: """Check debug is disabled on server-side""" assert (