-
Notifications
You must be signed in to change notification settings - Fork 54
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
feat: Semantic splitter #63
Conversation
Amazing @simjak |
getting the following after running
|
Solved this semantic router version |
Updated dependencies but still seeing the issue. |
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.
I did some small fixes and changes but seems to work well, good job! These changes have some braking changes with the encoder provider
in the API so I need to first migrate to that before merging.
Or we should set the new splitter config as non mandatory and just set a default? Perhaps you could add that in?
* Add support for queryig code interpreter * Fix formatting * Ensure the sandbox close is called on exceptions * Update service/code_interpreter.py Co-authored-by: Tomas Valenta <[email protected]> * Update service/code_interpreter.py Co-authored-by: Tomas Valenta <[email protected]> * Update service/router.py Co-authored-by: Tomas Valenta <[email protected]> * Update service/code_interpreter.py Co-authored-by: Tomas Valenta <[email protected]> * Add system prompt * Format code * Bump dependencies * Minor tweaks --------- Co-authored-by: Tomas Valenta <[email protected]>
Follow the walkthrough.ipynb for how to use