Skip to content

fix(chat): gpt-6-astra does not use temperature. - #6272

Open
ndbellew wants to merge 2 commits into
odysseus-dev:devfrom
ndbellew:fix/chatgpt-subscription-temperature
Open

ndbellew wants to merge 2 commits into
odysseus-dev:devfrom
ndbellew:fix/chatgpt-subscription-temperature

Conversation

@ndbellew

@ndbellew ndbellew commented Sep 11, 2026

Copy link
Copy Markdown

Summary

Prevents Odysseus from sending the unsupported temperature parameter when using the ChatGPT Subscription provider. This fixes HTTP 400 errors returned by subscription-backed models that reject temperature in the request payload.

Target branch

  • This PR targets dev, not main. All PRs land in dev; main is curated by the maintainer at each release. If your PR is on main by accident, click "Edit" on this PR and change the base.

Linked Issue

Fixes #6273

Type of Change

  • Bug fix (non-breaking — fixes a confirmed issue)
  • New feature (non-breaking — adds new behaviour)
  • Breaking change (changes or removes existing behaviour)
  • Refactor / cleanup (behaviour unchanged)
  • Documentation only
  • CI / tooling / configuration

Checklist

  • I searched open issues and open PRs — this is not a duplicate.
  • This PR targets dev
  • My changes are limited to the scope described above — no unrelated refactors or whitespace changes mixed in.
  • I actually ran the app (docker compose up) and verified the change works end-to-end.
  • I did not run the app/runtime validation and stated that gap in How to Test.

How to Test

  1. Configure a ChatGPT Subscription provider in Odysseus and select a model that rejects the temperature parameter.
  2. Send a normal chat message and confirm the request completes successfully instead of returning HTTP 400: Unsupported parameter: temperature.
  3. Verify non-ChatGPT providers and models that support temperature continue to receive the parameter normally.

Visual / UI changes — REQUIRED if you touched anything that renders

No visual or UI changes were made.

  • Screenshot or short clip of the change in the running app, attached below.
  • Style match
  • No new component patterns
  • I am not an LLM agent submitting a bulk PR.

Screenshots / clips

N/A — no UI or rendering changes.

@github-actions github-actions Bot added needs work PR description incomplete — please update before review needs runtime validation Runtime validation not attested — tick the app-run box after running it, or state the gap ready for review Description complete — ready for maintainer review and removed needs work PR description incomplete — please update before review needs runtime validation Runtime validation not attested — tick the app-run box after running it, or state the gap labels Sep 11, 2026
@ndbellew ndbellew changed the title gpt-6-astra does not use temperature. fix(chat): gpt-6-astra does not use temperature. Sep 11, 2026
@ndbellew

Copy link
Copy Markdown
Author

🙌🙌

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

ready for review Description complete — ready for maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The ChatGPT 6 Astra does not use Temperature

1 participant