Skip to content

fix: replace retired Qwen2.5 model in tutorial 30 - #486

Merged
bilgeyucel merged 1 commit into
mainfrom
fix/tutorial30-replace-retired-qwen25-model
Aug 28, 2026
Merged

fix: replace retired Qwen2.5 model in tutorial 30#486
bilgeyucel merged 1 commit into
mainfrom
fix/tutorial30-replace-retired-qwen25-model

Conversation

@julian-risch

@julian-risch julian-risch commented Aug 28, 2026

Copy link
Copy Markdown
Member

Related Issues

None. Found while investigating the nightly failure in run 33136680751; companion to deepset-ai/haystack-core-integrations#3866

Qwen/Qwen2.5-7B-Instruct + provider="together" routes to Together's Qwen2.5-7B-Instruct-Turbo, which Together retired from serverless. HF's API now reports that provider mapping as status=error, and the model is absent from router.huggingface.co/v1/models

Proposed Changes:

  • Swapped to Qwen/Qwen3.5-9B (same provider) — recent, small, cheap, live on together, and now consistent with the model used in the huggingface_api integration tests.
  • Added generation_kwargs={"extra_body": {"chat_template_kwargs": {"enable_thinking": False}}}. Qwen3.5 is a hybrid-thinking model; left on, it spends 8× the completion tokens on this tutorial's prompt (1292 vs 165)
  • Updated the prose model mention and the model name in the illustrative sample-response block

How did you test it?

Both tutorial 30 jobs pass on this PR

Notes for the reviewer

Checklist

  • I have read the contributors guidelines and the code of conduct
  • I have updated the related issue with new insights and changes — n/a, no issue
  • I added unit tests and updated the docstrings — n/a, tutorial notebook change
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Tutorial 30 calls `Qwen/Qwen2.5-7B-Instruct` through `provider="together"`,
which routes to Together's `Qwen2.5-7B-Instruct-Turbo`. Together retired
that model from serverless, so the HF provider mapping now reports
`status=error` and the model is gone from the router's model list.

The nightly run had two stacked causes, the first masking the second: it
ended on `402 Payment Required` ("You have depleted your monthly included
credits"), so the request never reached Together's model resolution and
the retired model surfaced only as a health-check warning. The 402 has
since cleared; the retired model had not.

Swap in `Qwen/Qwen3.5-9B` and disable thinking. Qwen3.5 is a hybrid
thinking model, and left on it spends ~8x the completion tokens (1292 vs
165 on this tutorial's prompt) and prepends a 4 KB reasoning trace, which
would make the tutorial's sample response unrepresentative.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@julian-risch
julian-risch force-pushed the fix/tutorial30-replace-retired-qwen25-model branch from e7d2a4d to eb47c47 Compare August 28, 2026 08:14
@julian-risch
julian-risch marked this pull request as ready for review August 28, 2026 08:24
@julian-risch
julian-risch requested a review from a team as a code owner August 28, 2026 08:24
@bilgeyucel
bilgeyucel merged commit da93f66 into main Aug 28, 2026
4 checks passed
@bilgeyucel
bilgeyucel deleted the fix/tutorial30-replace-retired-qwen25-model branch August 28, 2026 12:14
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