Skip to content
This repository was archived by the owner on Aug 22, 2025. It is now read-only.

Set temperature to None by default#406

Merged
jlowin merged 5 commits intomainfrom
temp-none
Feb 6, 2025
Merged

Set temperature to None by default#406
jlowin merged 5 commits intomainfrom
temp-none

Conversation

@jlowin
Copy link
Member

@jlowin jlowin commented Feb 6, 2025

o3 doesn't support temperature, so we need to set this to None to avoid passing through an illegal value.

@github-actions github-actions bot added breaking change The public API changes in a backwards-incompatible way enhancement Enhance an existing feature labels Feb 6, 2025
@jlowin jlowin merged commit 739b165 into main Feb 6, 2025
5 checks passed
@jlowin jlowin deleted the temp-none branch February 6, 2025 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

breaking change The public API changes in a backwards-incompatible way enhancement Enhance an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant