[yuanheng-zhao]

PLEASE FILL IN THE PR DESCRIPTION HERE.

Purpose

Resolves #4726

I didn't refactor or move the orders of the previous paths in StageConfigFactory.create_from_model, as we don't want to import significant changes which require a lot of testing. Instead, I added the user written deploy-config pipeline path (pipeline key in deploy yaml) at the end of StageConfigFactory.create_from_model, so that models fail for the other resolutions (e.g., model type none or generic, etc) will get into this path as an additional try.

Test Plan

Serve inclusionAI/Ming-omni-tts-0.5B

vllm-omni serve inclusionAI/Ming-omni-tts-0.5B \
    --deploy-config vllm_omni/deploy/ming_tts.yaml \
    --omni \
    --port 8091 \
    --enforce-eager

vllm-omni serve inclusionAI/Ming-omni-tts-0.5B \
    --omni \
    --port 8091 \
    --enforce-eager

Serve inclusionAI/Ming-omni-tts-16.8B-A3B

vllm-omni serve inclusionAI/Ming-omni-tts-16.8B-A3B --deploy-config vllm_omni/deploy/ming_tts_moe.yaml --omni --port 8091

vllm-omni serve inclusionAI/Ming-omni-tts-16.8B-A3B --omni --port 8091

vLLM Version: 0.23

vLLM-Omni Commit: on top of f881585

Test Result

I run serve and simple request on all the above four commands (Ming-omni-tts-0.5B and Ming-omni-tts-16.8B-A3B; with and without deploy-config arg). Servers get launched and configs get resolved correctly.

BEFORE SUBMITTING: read CONTRIBUTING.md and run the precheck-pr skill with the code agent for a self-check against project conventions.
(anything written below this line will be removed by GitHub Actions)

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[yuanheng-zhao]

PTAL @alex-jw-brooks

[alex-jw-brooks]

Thanks for the fix! Can you please add a regression test to tests/test_config_factory.py to cover this case? E.g.,

    def test_resolve_when_autodetect_resolves_none(self):
        """Regression test for: https://github.com/vllm-project/vllm-omni/issues/4726"""
        deploy_path = get_deploy_config_path("ming_tts.yaml")
        resolved_config = StageConfigFactory.create_from_model(
            model="inclusionAI/Ming-omni-tts-0.5B",
            deploy_config_path=deploy_path,
        )
        assert resolved_config is not None
        assert len(resolved_config) > 0

Already checked to make sure it fails on main and passes on your branch

[hsliuustc0106]

Bugfix looks correct.