Skip to content

feat(agent): The SQL execution result of the DataScientist can be passed to the next agent#2654

Merged
Aries-ckt merged 1 commit into
eosphoros-ai:mainfrom
vnicers:feat_dsagent
Apr 29, 2025
Merged

feat(agent): The SQL execution result of the DataScientist can be passed to the next agent#2654
Aries-ckt merged 1 commit into
eosphoros-ai:mainfrom
vnicers:feat_dsagent

Conversation

@vnicers

@vnicers vnicers commented Apr 25, 2025

Copy link
Copy Markdown
Contributor

Description

By default,the sql execution result of DataScientistis Agent displayed only in vis chart, if you want to use the Summary Agent to summarize the results of DataScientistis Agent, you can only analyze whether the SQL execution is correct, but cannot analyze the data queried by the SQL, which does not meet our expectations,I pass the SQL query results to the Content node. Other agents can do the processing

How Has This Been Tested?

image

Checklist:

  • My code follows the style guidelines of this project
  • I have already rebased the commits and make the commit message conform to the project standard.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules

- Add data_df content to the output if not empty- Convert data_df to JSON format and include in the output
- Update content to a comprehensive JSON representation
@github-actions github-actions Bot added agent Module: agent enhancement New feature or request labels Apr 25, 2025

@fangyinc fangyinc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Aries-ckt Aries-ckt 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.

Thanks for your contribution.LGTM

@Aries-ckt Aries-ckt merged commit b901cbc into eosphoros-ai:main Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent Module: agent enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants