Skip to content

refactor: inject scheduler and EPLB options across runtime boundaries. - #2738

Open
liutongxuan wants to merge 3 commits into
xLLM-AI:mainfrom
liutongxuan:features/refactor_options3
Open

liutongxuan wants to merge 3 commits into
xLLM-AI:mainfrom
liutongxuan:features/refactor_options3

Conversation

@liutongxuan

Copy link
Copy Markdown
Collaborator

Description

Related Issues

Change Type

  • Bug fix
  • New feature
  • Performance improvement
  • Refactor
  • Documentation
  • Test
  • Build or CI

Pull Request Checklist

Thank you for contributing to xLLM. Before requesting review, please make sure the following items are complete.

PR Title and Commit Messages

  • The PR title and each commit message follow the xLLM commit format: <type>: <subject>.

Allowed types: feat, bugfix, docs, test, refactor, chore, style, revert, perf, model, build, release.
The subject should use clear English, start with a verb, include at least 4 words, and end with ..

Pre-commit Checks

  • I have installed pre-commit by running pip install pre-commit or an equivalent command.
  • I have installed the hooks with pre-commit install.
  • I have run pre-commit run --all-files and fixed any reported issues.

If you are unsure how to set up pre-commit, see the pre-commit documentation.

Self Review

  • I have self-reviewed the code according to .agents/skills/code-review/references/custom-code-style.md, especially code written or assisted by AI.
  • I have rebased this PR onto the latest main branch.

Build and Test Coverage

  • Tests have been added or updated as needed.
  • CUDA: python setup.py build test has passed on a CUDA machine.
  • NPU: python setup.py build test has passed on an NPU machine.
  • MLU: python setup.py build test has passed on an MLU machine.

Reviewer Notes

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-13T21:42:57.719728Z
We couldn’t complete this review. Comment @codex review to try again.
79a6b8c PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@liutongxuan liutongxuan changed the title Features/refactor options3 refactor: inject scheduler and EPLB options across runtime boundaries. Nov 13, 2026
Serialize producer-owned expert loads and prepare tokens without consulting process-wide enablement. Keep tensor validation and shared-memory capacity checks at the transport boundary.

Cover LLM and VLM payloads, token-only results, empty metadata, and conflicting global settings in both transports.
Project mix-batch enablement and effective KV split into SchedulerOptions at the LLM and VLM master boundaries. Batch-mode selection then depends on its input while preserving chunking, parallel-layout, speculative-decoding, and PD role rules.

Exercise conflicting process settings and migrate existing mode-specific tests to explicit inputs without changing chunk-size resolution.
Resolve worker enablement, redundant expert count, and decode-only measurement once at the LLM master. Carry this snapshot through in-process and spawned worker bootstrap, collective communicators, and model context into native MoE layers and the Python bridge.

Preserve load buffers for both target and draft workers while keeping the EPLB executor target-only. Keep expert parallel degree and rank-table configuration in their existing communication bootstrap paths.

Validate appended spawn arguments at the process boundary and retain legacy defaults. Cover conflicting globals, snapshot copies, native expert storage, Python configuration, overlap metadata, and target/draft initialization.
@liutongxuan
liutongxuan force-pushed the features/refactor_options3 branch from 79a6b8c to 9cc6e6e Compare November 15, 2026 08:04

This branch has not been deployed

No deployments
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.

1 participant