Abhinav 2ba3ce81a6 fix(openai): make GPT-5 temperature validation case-insensitive (#34012) ...

Fixed a bug where GPT-5 temperature validation was case-sensitive,
causing issues when users
specified Azure deployment names or model names in uppercase (e.g.,
`"GPT-5-2025-01-01"`, `"GPT-5-NANO"`). The validation now correctly
handles model names regardless of case.

  Changes made:
- Updated `validate_temperature()` method in `BaseChatOpenAI` to perform
case-insensitive
  model name comparisons
- Updated `_get_encoding_model()` method to use case-insensitive checks
for tiktoken encoder
  selection
- Added comprehensive unit tests to verify case-insensitive behavior
with various case
  combinations

  **Issue:** Fixes #34003

  **Dependencies:** None

  **Test Coverage:**
  - All existing tests pass
- New test `test_gpt_5_temperature_case_insensitive` covers uppercase,
lowercase, and
  mixed-case model names
- Tests verify both non-chat GPT-5 models (temperature removed) and chat
models (temperature
  preserved)
  - Lint and format checks pass (`make lint`, `make format`)

---------

Co-authored-by: Mason Daugherty <github@mdrxy.com>

2025-11-23 20:17:03 -05:00

..

cassettes

feat: The response.incomplete event is not handled when using stream_mode=['messages'] (#33871)

2025-11-07 09:46:11 -05:00

integration_tests

chore(openai): skip Azure text completions tests (#34021)

2025-11-19 09:29:12 -05:00

unit_tests

fix(openai): make GPT-5 temperature validation case-insensitive (#34012)

2025-11-23 20:17:03 -05:00

__init__.py

…

conftest.py

docs: more standardization (#33124)

2025-09-25 20:46:20 -04:00