Skip to content

Conversation

shen-shanshan
Copy link
Collaborator

@shen-shanshan shen-shanshan commented Aug 8, 2025

What this PR does / why we need it?

Add test for outlines backend for structured output in CI.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Tests have all passed with:

pytest -sv tests/e2e/singlecard/test_guided_decoding.py

Copy link

github-actions bot commented Aug 8, 2025

👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:‌‌

  • A PR should do only one thing, smaller PRs enable faster reviews.
  • Every PR should include unit tests and end-to-end tests ‌to ensure it works and is not broken by other future PRs.
  • Write the commit message by fulfilling the PR description to help reviewer and future developers understand.

If CI fails, you can run linting and testing checks locally according Contributing and Testing.

Copy link

codecov bot commented Aug 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.69%. Comparing base (60ac4fb) to head (58de558).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2283      +/-   ##
==========================================
- Coverage   77.70%   77.69%   -0.02%     
==========================================
  Files         132      132              
  Lines       17521    17521              
==========================================
- Hits        13615    13613       -2     
- Misses       3906     3908       +2     
Flag Coverage Δ
unittests 77.69% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@shen-shanshan shen-shanshan force-pushed the so-2 branch 7 times, most recently from 5e24c92 to 202dabb Compare August 15, 2025 01:29
Signed-off-by: shen-shanshan <467638484@qq.com>
Signed-off-by: shen-shanshan <467638484@qq.com>
@shen-shanshan
Copy link
Collaborator Author

@wangxiyuan The CI has passed, just adding a new backend test.

@wangxiyuan wangxiyuan merged commit 0767d51 into vllm-project:main Aug 25, 2025
16 checks passed
wangxiaoteng888 pushed a commit to LCAIZJ/vllm-ascend that referenced this pull request Sep 25, 2025
…d output in CI (vllm-project#2283)

### What this PR does / why we need it?
Add test for `outlines` backend for structured output in CI.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?

Tests have all passed with:

```bash
pytest -sv tests/e2e/singlecard/test_guided_decoding.py
```

- vLLM version: v0.10.0
- vLLM main:
vllm-project/vllm@5341565

---------

Signed-off-by: shen-shanshan <467638484@qq.com>
chopper0126 pushed a commit to chopper0126/vllm-ascend that referenced this pull request Sep 26, 2025
…d output in CI (vllm-project#2283)

### What this PR does / why we need it?
Add test for `outlines` backend for structured output in CI.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?

Tests have all passed with:

```bash
pytest -sv tests/e2e/singlecard/test_guided_decoding.py
```

- vLLM version: v0.10.0
- vLLM main:
vllm-project/vllm@5341565

---------

Signed-off-by: shen-shanshan <467638484@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants