-
Notifications
You must be signed in to change notification settings - Fork 9k
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
Chore/remove python dependencies selector #7494
Conversation
Why remove list python dependencies funciton? |
This is not required anymore. If you add things in the requirements.txt, it should be able to be imported automatically. |
The previous code node automatically imported the standard library (such as datetime and time), so the user's code did not need to explicitly declare import statements. However, after updating to this version, the code node must explicitly declare import statements, otherwise the user's code will report a "not defined error". In this case, how can the running business code be updated? |
Checklist:
Important
Please review the checklist below before submitting your pull request.
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint godsDescription
Selecting python dependencies manually is no longer required due to the update of dify-sandbox
Fixes
Type of Change
Testing Instructions
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