Skip to content

chore: import DaytonaDataAnalysisTool#5

Merged
mislavivanda merged 2 commits into
mainfrom
chore/import-langchain-data-analysis
Jun 26, 2026
Merged

chore: import DaytonaDataAnalysisTool#5
mislavivanda merged 2 commits into
mainfrom
chore/import-langchain-data-analysis

Conversation

@mislavivanda

@mislavivanda mislavivanda commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.


Summary by cubic

Import the langchain-daytona-data-analysis package to add DaytonaDataAnalysisTool for running Python data analysis in a secure, persistent Daytona sandbox. Adds file handling, pip installs, optional result callbacks, and ships with docs, a notebook, and tests.

  • New Features

    • DaytonaDataAnalysisTool: run Python in a persistent Daytona sandbox; auto-prints the last expression when print(...) is missing.
    • File ops: upload, download, remove; tool lists uploaded files in its description.
    • Sandbox control and packages: install_python_packages(...), get_sandbox(), close().
    • Optional on_result callback for ExecutionArtifacts (stdout, charts).
  • Dependencies

    • New package at packages/langchain-data-analysis published as langchain-daytona-data-analysis.
    • Runtime deps: daytona ^0.113.2, langchain-core >=0.3.15; Python >=3.9.
    • CI: pin to Python 3.12 for obstore wheels (transitive via daytona).
    • Auth via DAYTONA_API_KEY or pass daytona_api_key.

Written for commit 9cf5099. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

20 issues found across 17 files

Tip: instead of fixing issues one by one fix them all with cubic

Re-trigger cubic

Comment thread packages/langchain-data-analysis/scripts/lint_imports.sh
Comment thread packages/langchain-data-analysis/docs/tools.ipynb
Comment thread packages/langchain-data-analysis/tests/unit_tests/test_tools.py
Comment thread packages/langchain-data-analysis/Makefile
Comment thread packages/langchain-data-analysis/docs/tools.ipynb
Signed-off-by: Mislav Ivanda <mislavivanda454@gmail.com>
Signed-off-by: Mislav Ivanda <mislavivanda454@gmail.com>
@mislavivanda mislavivanda force-pushed the chore/import-langchain-data-analysis branch from 8db46fb to 9cf5099 Compare June 26, 2026 23:34
@mislavivanda mislavivanda merged commit 6fad62e into main Jun 26, 2026
9 checks passed
@mislavivanda mislavivanda deleted the chore/import-langchain-data-analysis branch June 26, 2026 23:44
mislavivanda added a commit that referenced this pull request Jul 6, 2026
Signed-off-by: Mislav Ivanda <mislavivanda454@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant