-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
improve: code upgrade #4231
improve: code upgrade #4231
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.
LGTM
how can i install my |
I guess it should be in the docker container of docker-sandbox. I just checked pip list in the container and httpx, jinja2 are alreay there. |
Just add them into sandbox/dependencies/python-requirements.txt, but not all dependencies could work as expected while testing, just try it carefully I'll solve them in months. |
Great job! |
hope to support libraries like pandas and numpy as soon as possible to give AI workflows the ability to process data. |
Description
Support Dependenices
Support Networking
Others
Fixes # (issue)
Type of Change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Suggested Checklist:
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint godsoptional
I have made corresponding changes to the documentationoptional
I have added tests that prove my fix is effective or that my feature worksoptional
New and existing unit tests pass locally with my changes