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 an option to auto-delete temp files created by LocalCommandLineExecutor #4380

Open
afourney opened this issue Nov 26, 2024 · 1 comment
Assignees
Labels
code-execution execute generated code good first issue Good for newcomers proj-core
Milestone

Comments

@afourney
Copy link
Member

What feature would you like to be added?

LocalCommandLineExecutor leaves a lot of debris on disk like:

tmp_code_84c00a94989f5048ee254cb2e583fc8e75b66c0afef4a7a76c8901a4eb2f5bc5.py
tmp_code_95c44f5140612045a5fb8c5ccad956a2ae4afb4a028201cf836fbb55c09c72f1.py

These files are executed once, and never referred to again.

Please add an (on by default) option to automatically clean up temporary files.

Why is this needed?

Explained above.

@afourney afourney added code-execution execute generated code proj-core labels Nov 26, 2024
@afourney afourney added this to the 0.4.0 milestone Nov 26, 2024
@MohMaz MohMaz added good first issue Good for newcomers and removed needs-triage labels Nov 26, 2024
@jackgerrits
Copy link
Member

We don't need to do this work in the framework, using standard python tempdir tooling should work fine: https://github.com/microsoft/autogen/blob/prefix_sub/python/packages/autogen-core/tests/execution/test_commandline_code_executor.py#L92-L93

@jackgerrits jackgerrits removed their assignment Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-execution execute generated code good first issue Good for newcomers proj-core
Projects
None yet
Development

No branches or pull requests

4 participants