Skip to content

Conversation

gaiksaya
Copy link
Member

@gaiksaya gaiksaya commented Oct 1, 2025

Description

Add default values as per description for release notes

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

description: 'Max number of tokens to use for supplied model id, defaults to 2000',
trim: true
trim: true,
defaultValue: 2000
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make it 4000 to be on the safer side.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Signed-off-by: Sayali Gaikawad <[email protected]>
Copy link

codecov bot commented Oct 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.57%. Comparing base (efb9953) to head (2bc3965).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5747   +/-   ##
=======================================
  Coverage   96.57%   96.57%           
=======================================
  Files         405      405           
  Lines       18506    18506           
=======================================
  Hits        17872    17872           
  Misses        634      634           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gaiksaya gaiksaya merged commit e139fc3 into opensearch-project:main Oct 1, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in Engineering Effectiveness Board Oct 1, 2025
@gaiksaya gaiksaya deleted the add-defaults branch October 1, 2025 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants