mirror of
https://github.com/LTTLabsOSS/markbench-tests.git
synced 2026-01-09 22:18:00 -05:00
Please linter, less problems more solutions
This commit is contained in:
@@ -94,9 +94,9 @@ def run_benchmark(process_name, command_to_run):
|
|||||||
|
|
||||||
# Log any output and error from the Steam process
|
# Log any output and error from the Steam process
|
||||||
if stdout:
|
if stdout:
|
||||||
logging.info(f"Steam Output: %s", stdout)
|
logging.info("Steam Output: %s", stdout)
|
||||||
if stderr:
|
if stderr:
|
||||||
logging.error(f"Steam Error: %s", stderr)
|
logging.error("Steam Error: %s", stderr)
|
||||||
|
|
||||||
# Now, return the process object
|
# Now, return the process object
|
||||||
return proc
|
return proc
|
||||||
|
|||||||
@@ -43,7 +43,7 @@ def delete_old_scores(file):
|
|||||||
for file in files:
|
for file in files:
|
||||||
try:
|
try:
|
||||||
os.remove(file)
|
os.remove(file)
|
||||||
logging.info(f"Deleted old score file: {file}")
|
logging.info("Deleted old score file: %s", file)
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
print(f"Error deleting file {file}: {e}")
|
print(f"Error deleting file {file}: {e}")
|
||||||
|
|
||||||
@@ -73,7 +73,7 @@ def is_process_running(process_name):
|
|||||||
|
|
||||||
def wait_for_benchmark_process(test_name, process_name, timeout=60):
|
def wait_for_benchmark_process(test_name, process_name, timeout=60):
|
||||||
"""Wait for the benchmark game process to start and then finish."""
|
"""Wait for the benchmark game process to start and then finish."""
|
||||||
logging.info(f"Waiting for benchmark process '{process_name}' to start...")
|
logging.info("Waiting for benchmark process '%s' to start...", process_name)
|
||||||
|
|
||||||
start_time = time.time()
|
start_time = time.time()
|
||||||
|
|
||||||
@@ -81,15 +81,15 @@ def wait_for_benchmark_process(test_name, process_name, timeout=60):
|
|||||||
# Check if the benchmark process is running
|
# Check if the benchmark process is running
|
||||||
process = is_process_running(process_name)
|
process = is_process_running(process_name)
|
||||||
if process:
|
if process:
|
||||||
logging.info(f"{test_name} has started. Waiting for it to finish...")
|
logging.info("%s has started. Waiting for it to finish...", test_name)
|
||||||
process.wait() # This will block until the process finishes
|
process.wait() # This will block until the process finishes
|
||||||
logging.info("Benchmark has finished.")
|
logging.info("Benchmark has finished.")
|
||||||
break
|
break
|
||||||
|
|
||||||
# If we exceed the timeout, break out of the loop and log an error
|
# If we exceed the timeout, break out of the loop and log an error
|
||||||
if time.time() - start_time > timeout:
|
if time.time() - start_time > timeout:
|
||||||
logging.error(f"Timeout reached while waiting for process '%s'.", process_name)
|
logging.error("Timeout reached while waiting for process '%s'.", process_name)
|
||||||
raise TimeoutError(f"Process '%s' did not start within the expected time. Is the game configured for DX12?", process_name)
|
raise TimeoutError("Process '%s' did not start within the expected time. Is the game configured for DX12?", process_name)
|
||||||
|
|
||||||
# Wait for 1 second before checking again
|
# Wait for 1 second before checking again
|
||||||
time.sleep(1)
|
time.sleep(1)
|
||||||
|
|||||||
@@ -23,4 +23,4 @@ def get_args() -> any:
|
|||||||
"--kerasHost", dest="keras_host", help="Host for Keras OCR service", required=True)
|
"--kerasHost", dest="keras_host", help="Host for Keras OCR service", required=True)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
"--kerasPort", dest="keras_port", help="Port for Keras OCR service", required=True)
|
"--kerasPort", dest="keras_port", help="Port for Keras OCR service", required=True)
|
||||||
return parser.parse_args()
|
return parser.parse_args()
|
||||||
|
|||||||
@@ -90,4 +90,4 @@ def get_args() -> any:
|
|||||||
"--kerasHost", dest="keras_host", help="Host for Keras OCR service", required=True)
|
"--kerasHost", dest="keras_host", help="Host for Keras OCR service", required=True)
|
||||||
parser.add_argument(
|
parser.add_argument(
|
||||||
"--kerasPort", dest="keras_port", help="Port for Keras OCR service", required=True)
|
"--kerasPort", dest="keras_port", help="Port for Keras OCR service", required=True)
|
||||||
return parser.parse_args()
|
return parser.parse_args()
|
||||||
|
|||||||
Reference in New Issue
Block a user