-
Notifications
You must be signed in to change notification settings - Fork 66
feat(ia_utils): accept kwargs at assert text similarity #441
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR refactors text similarity functions to accept keyword arguments (**kwargs) instead of specific parameters, providing greater flexibility when calling these functions with different models and configurations.
Key changes:
- Replaced specific function parameters (model_name, azure) with **kwargs across text similarity and OpenAI request functions
- Updated function signatures and documentation to reflect the kwargs pattern
- Added test coverage for the new kwargs functionality
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| toolium/utils/ai_utils/text_similarity.py | Modified all text similarity functions to accept **kwargs instead of specific parameters; updated docstrings accordingly |
| toolium/utils/ai_utils/openai.py | Refactored openai_request to use **kwargs pattern for model_name and azure parameters |
| toolium/test/utils/ai_utils/test_text_similarity.py | Added test cases validating kwargs propagation through assert_text_similarity |
| docs/ai_utils.rst | Updated documentation examples to show model_name and azure parameter usage |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.