Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Removed monkey patch, OpenShift AI now works with max_completion_tokens #685

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

shawn-hurley
Copy link
Contributor

fixes #627

@shawn-hurley shawn-hurley force-pushed the bugfix/remove-monkey-patch branch from 3e6383c to 523da5a Compare February 24, 2025 23:02
@shawn-hurley
Copy link
Contributor Author

removed all the code to not do the validate_environment. Not all models use pydantic model validation.

@shawn-hurley shawn-hurley force-pushed the bugfix/remove-monkey-patch branch from 523da5a to 0d5af37 Compare February 25, 2025 21:09
Signed-off-by: Shawn Hurley <shawn@hurley.page>
@shawn-hurley shawn-hurley force-pushed the bugfix/remove-monkey-patch branch from 0d5af37 to 101d32c Compare February 25, 2025 21:11
Copy link
Contributor

@JonahSussman JonahSussman left a comment

Choose a reason for hiding this comment

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

ACK

@JonahSussman JonahSussman changed the title 🐛 remove monkey patch, openshift ai now works with max_completion_tokens 🐛 Removed monkey patch, OpenShift AI now works with max_completion_tokens Feb 25, 2025
@shawn-hurley shawn-hurley merged commit 4363ded into konveyor:main Feb 25, 2025
13 checks passed
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.

Monkey patch _default_params and _get_request_payload only if needed
3 participants