Skip to content

Raise runner prepare defaults#601

Merged
pan-x-c merged 1 commit into
agentscope-ai:mainfrom
shiweijiezero:perf/runner-prepare-defaults
Jul 7, 2026
Merged

Raise runner prepare defaults#601
pan-x-c merged 1 commit into
agentscope-ai:mainfrom
shiweijiezero:perf/runner-prepare-defaults

Conversation

@shiweijiezero

Copy link
Copy Markdown
Contributor

Summary

  • Raise the default runner_prepare_concurrency from 8 to 128.
  • Increase runner_prepare_max_retries from 2 to 3.

Motivation

The current default concurrency of 8 can slow down large workflow-runner cold starts. In DLC profiling, each runner prepare can take around 70 seconds, so preparing many runners in batches of 8 can noticeably delay startup and GPU utilization.

The 128 value keeps a conservative default guardrail for mass startup. If maintainers prefer simpler behavior, keeping only the retry mechanism may also be enough, since transient runner-prepare failures are already retried.

Checks

  • python -m py_compile trinity/common/config.py

@pan-x-c

pan-x-c commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

/unittest-module-explorer

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

unittest: Run #1812

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
74 73 1 0 0 0 0 19m 13s

Some tests failed!

Name Failure Message
❌ tests/explorer/explorer_test.py::TestExplorerGSM8KRULERNoEval::test_explorer The test failed in the call phase due to an assertion error

Github Test Reporter by CTRF 💚

@pan-x-c

pan-x-c commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

/unittest-pattern-TestExplorerGSM8KRULERNoEval

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

unittest: Run #1813

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
1 1 0 0 0 0 0 3m 28s

🎉 All tests passed!

Github Test Reporter by CTRF 💚

@pan-x-c pan-x-c merged commit 8f4fb93 into agentscope-ai:main Jul 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants