-
Notifications
You must be signed in to change notification settings - Fork 745
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: Integrate e2b. #1284
feat: Integrate e2b. #1284
Conversation
Thanks @willshang76 , for the dependency issue, PR raised here: AgentOps-AI/agentops#556 |
…ation # Conflicts: # poetry.lock
# Conflicts: # poetry.lock
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.
Thanks @willshang76 ! Left some comments below
code_type (str): The type of code to execute (e.g., 'python', | ||
'bash'). |
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.
use Literal
instead of str
?
code_type: Literal["python", "py3", "python3", "py", "shell", "bash", "sh", "java", "javascript", "js", "r"]
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.
Actually the base class defined this abstract method as type str for this parameter. Therefore I think that we could leave it as this for this PR, we could create a separate issue to refactor this part. How do you think?
What is |
e2b-code-interpreter |
There is a |
The e2b-code-interpreter package actually depends on the e2b package. When I did poetry add e2b-code-interpreter; it also gets the e2b. |
It is `e2d` not `e2b`
…On Thu 12. Dec 2024 at 15:39, Will Shang ***@***.***> wrote:
There is a e2d as well. I think it is a typo
The e2b-code-interpreter package actually depends on the e2b package. When
I did poetry add e2b-code-interpreter; it also gets the e2b.
—
Reply to this email directly, view it on GitHub
<#1284 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFUJTYB7FSFQPJM2MUADFTD2FE4UZAVCNFSM6AAAAABTDTNKJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZYGAZTCMRTGI>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Thanks @lightaime , @willshang76 , we were waiting for AgentOps to update their dependency to solve the conflict, I have updated the dependency now |
Description
Integrate the e2b interpreter into camel. https://github.com/e2b-dev/
Motivation and Context
close #1245
Types of changes
What types of changes does your code introduce? Put an
x
in all the boxes that apply:Implemented Tasks
Checklist
Go over all the following points, and put an
x
in all the boxes that apply.If you are unsure about any of these, don't hesitate to ask. We are here to help!