Skip to content

feat: data analysis demo - #29

Merged
cuericlee merged 28 commits into
bytedance:mainfrom
marchhao:feature/data_analysis_demo
Dec 12, 2025
Merged

feat: data analysis demo#29
cuericlee merged 28 commits into
bytedance:mainfrom
marchhao:feature/data_analysis_demo

Conversation

@marchhao

Copy link
Copy Markdown
Contributor

feat: data analysis demo

class LanceDBManager:
def __init__(self):
# Configuration from environment
self.lancedb_uri = os.getenv("LANCEDB_URI", "s3://emr-serverless-sdk/lance_catalog/default/imdb_top_1000")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

reference to tos uri to avoid oversea latency?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

目前还改不了,开源lancedb只支持Valid schemes: S3、OSS等,还不支持TOS,已提需求

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

文件是在TOS上,仅使用S3协议进行访问

@@ -0,0 +1,5 @@
# Ark (OpenAI compatible)
ARK_API_KEY=

@cuericlee cuericlee Dec 11, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

using MODEL_AGENT_API_KEY instead

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已修改

ARK_API_KEY=

# LAS Seedance Online
LAS_API_KEY= No newline at end of file

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  1. add one more api_key here, TOOL_LAS_API_KEY firstly
    https://github.com/volcengine/veadk-python/blob/main/config.yaml.full#L67
  2. and figure out a way to using ak/sk to fetch apikey instead of env in the future.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已修改为TOOL_LAS_API_KEY,ak/sk to fetch apikey instead of env这个我后续添加支持

return response

# 创建带记忆的 Agent
model_name = os.getenv("AGENT_MODEL_NAME", "doubao-seed-1-6-251015") # 默认使用更主流的豆包模型

@cuericlee cuericlee Dec 11, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

AGENT_MODEL_NAME -》 MODEL_AGENT_NAME
AGENT_MODEL_API_KEY -〉MODEL_AGENT_API_KEY

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已修改

@marchhao

Copy link
Copy Markdown
Contributor Author

自验证:
image
image
image
image

@marchhao

Copy link
Copy Markdown
Contributor Author

agentkit.yaml中已经移除提交

@bytedance bytedance deleted a comment from marchhao Dec 11, 2025
@@ -0,0 +1,2 @@
# Package marker for tools modules

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

don't use agentkit_deploy folder

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已修改

# 处理当前用户输入
response = super().run(input_text, **kwargs)
# 将当前交互保存到记忆
self.memory_manager.add_message(session_id=session_id, role="user", content=input_text)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why do you need add message in case you already has shorttermmeory?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已修改

@cuericlee cuericlee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

/lgtm

@cuericlee
cuericlee merged commit 688df97 into bytedance:main Dec 12, 2025
1 check passed
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.

2 participants