Check terminal toolkit initial environment setting#3818
Check terminal toolkit initial environment setting#3818Abishek-Newar wants to merge 9 commits intocamel-ai:masterfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
bytecii
left a comment
There was a problem hiding this comment.
Thanks. Left some comments
|
Should we made these runtimes configurable from the initialzation of terminal toolkit? Having all the runtimes installed by default may be a bit too heavy. I would prefer having python runtime by default but the other ones are optional in the We can enable them all in eigent product. Do we have an idea what kind of tasks we need go and java. Also do we have an idea how much extra space will it take if we want to bundle all the language runtimes into the package? @Wendong-Fan @bytecii WDYT? |
bytecii
left a comment
There was a problem hiding this comment.
Yea, let's maybe add a config to for example configure whether we need to include each language
|
@bytecii , can you please elaborate what type of changes should i do further? |
For example, for the TerminalToolkit initialization add args to something like allowed runtime etc. |
bytecii
left a comment
There was a problem hiding this comment.
LGTM after resolving the my comments.
|
Hi @Abishek-Newar. Thanks a lot for the contribution! It would be great to add an example like https://github.com/camel-ai/camel/blob/master/examples/toolkits/terminal_toolkit.py to demonstrate how to use these new language runtimes |
Description
Add Go and Java auto-install support for the terminal toolkit, as discussed in eigent-ai/eigent#932.
When the terminal toolkit initializes, it now automatically detects and downloads Go and Java runtimes if they aren't already installed:
go.dev/dl/(pinned to v1.23.6)~/.camel/runtimes/go/and~/.camel/runtimes/java/Changes
Closes : #932
closes: #3817
Checklist
Fixes #issue-numberin the PR description (required)pyproject.tomlanduv lock