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

Add support for querying code interpreter #66

Merged
merged 11 commits into from
Feb 29, 2024
Merged

Conversation

homanp
Copy link
Contributor

@homanp homanp commented Feb 28, 2024

What does this PR do?

Adds support for running code interpreter on structured docs.

Fixes #60

@homanp homanp added the enhancement New feature or request label Feb 28, 2024
@homanp homanp self-assigned this Feb 28, 2024
Copy link
Collaborator

@ValentaTomas ValentaTomas left a comment

Choose a reason for hiding this comment

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

I recommend updating to e2b SDK 0.14.7.

service/router.py Outdated Show resolved Hide resolved
@ValentaTomas ValentaTomas changed the title Add support for queryig code interpreter Add support for querying code interpreter Feb 28, 2024
@ValentaTomas
Copy link
Collaborator

ValentaTomas commented Feb 28, 2024

I improved the sandbox lifecycle handling with a0a1cf3 to ensure the sandbox is not left hanging where there is an exception (for example when there is a problem uploading files).

homanp and others added 8 commits February 28, 2024 18:06
Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>
Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>
Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>
Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>
@homanp homanp merged commit 9bcefb3 into main Feb 29, 2024
3 checks passed
@homanp
Copy link
Contributor Author

homanp commented Feb 29, 2024

Thank you for support @ValentaTomas ❤️

homanp added a commit to simjak/super-rag that referenced this pull request Feb 29, 2024
* 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 <valenta.and.thomas@gmail.com>

* Update service/code_interpreter.py

Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>

* Update service/router.py

Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>

* Update service/code_interpreter.py

Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>

* Add system prompt

* Format code

* Bump dependencies

* Minor tweaks

---------

Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>
homanp added a commit that referenced this pull request Mar 2, 2024
* feat: Semantic Spliter + minor improvements

* chore: testing

* fix: BaseDocumentChunk output fix

* feat: Combined chunk title with chunk content

* chore: Updated semantic router version

* chore: Updated semantic router version

* Small tweaks and bug fixes

* Update README

* chore: Merging

* chore: Testing

* chore: Minor improvements

* Add support for querying code interpreter (#66)

* 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 <valenta.and.thomas@gmail.com>

* Update service/code_interpreter.py

Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>

* Update service/router.py

Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>

* Update service/code_interpreter.py

Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>

* Add system prompt

* Format code

* Bump dependencies

* Minor tweaks

---------

Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>

* Minor tweaks

---------

Co-authored-by: Ismail Pelaseyed <homanp@gmail.com>
Co-authored-by: Tomas Valenta <valenta.and.thomas@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Computational Q&A
2 participants