Skip to content

Conversation

@bhimrazy
Copy link
Collaborator

What does this PR do ?

Fixes #547

Introduce a new reasoning_effort field to the ChatCompletionRequest model and implement corresponding tests to validate its functionality in the API responses.

@codecov
Copy link

codecov bot commented Jun 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85%. Comparing base (1ca73c5) to head (73661f0).
Report is 1 commits behind head on main.

Additional details and impacted files
@@         Coverage Diff         @@
##           main   #548   +/-   ##
===================================
  Coverage    85%    85%           
===================================
  Files        38     38           
  Lines      2901   2902    +1     
===================================
+ Hits       2479   2480    +1     
  Misses      422    422           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Borda Borda requested a review from aniketmaurya June 16, 2025 07:19
@aniketmaurya aniketmaurya merged commit e03ba13 into Lightning-AI:main Jun 16, 2025
21 checks passed
@bhimrazy bhimrazy deleted the upd/openai-spec-to-support-reasoning_effort branch June 16, 2025 07:37
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.

Add support for reasoning_effort in OpenAI spec for chat request

3 participants