mirror of
https://github.com/comfyanonymous/ComfyUI.git
synced 2025-09-14 05:25:23 +00:00
Add more leeway on async tests for Windows CI
This commit is contained in:
@@ -345,8 +345,8 @@ class TestAsyncNodes:
|
|||||||
dynamic_async = g.node("TestDynamicAsyncGeneration",
|
dynamic_async = g.node("TestDynamicAsyncGeneration",
|
||||||
image1=image1.out(0),
|
image1=image1.out(0),
|
||||||
image2=image2.out(0),
|
image2=image2.out(0),
|
||||||
num_async_nodes=3,
|
num_async_nodes=5,
|
||||||
sleep_duration=0.2)
|
sleep_duration=0.4)
|
||||||
g.node("SaveImage", images=dynamic_async.out(0))
|
g.node("SaveImage", images=dynamic_async.out(0))
|
||||||
|
|
||||||
start_time = time.time()
|
start_time = time.time()
|
||||||
@@ -354,7 +354,7 @@ class TestAsyncNodes:
|
|||||||
elapsed_time = time.time() - start_time
|
elapsed_time = time.time() - start_time
|
||||||
|
|
||||||
# Should execute async nodes in parallel within dynamic prompt
|
# Should execute async nodes in parallel within dynamic prompt
|
||||||
assert elapsed_time < 0.5, f"Dynamic async execution took {elapsed_time}s"
|
assert elapsed_time < 1.0, f"Dynamic async execution took {elapsed_time}s"
|
||||||
assert result.did_run(dynamic_async)
|
assert result.did_run(dynamic_async)
|
||||||
|
|
||||||
def test_async_resource_cleanup(self, client: ComfyClient, builder: GraphBuilder):
|
def test_async_resource_cleanup(self, client: ComfyClient, builder: GraphBuilder):
|
||||||
|
@@ -574,7 +574,8 @@ class TestExecution:
|
|||||||
|
|
||||||
# Similar to the previous test, expect parallel execution of the sleep nodes
|
# Similar to the previous test, expect parallel execution of the sleep nodes
|
||||||
# which should complete in less than the sum of all sleeps
|
# which should complete in less than the sum of all sleeps
|
||||||
assert elapsed_time < 10.0, f"Expansion execution took {elapsed_time}s, expected less than 5.5s"
|
# Lots of leeway here since Windows CI is slow
|
||||||
|
assert elapsed_time < 13.0, f"Expansion execution took {elapsed_time}s"
|
||||||
|
|
||||||
# Verify the parallel sleep node executed
|
# Verify the parallel sleep node executed
|
||||||
assert result.did_run(parallel_sleep), "ParallelSleep node should have run"
|
assert result.did_run(parallel_sleep), "ParallelSleep node should have run"
|
||||||
|
Reference in New Issue
Block a user