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 new codexgraph_agent #552

Merged
merged 5 commits into from
Jul 29, 2024
Merged

Conversation

laptype
Copy link
Contributor

@laptype laptype commented Jul 29, 2024

Change Summary

add new codexgraph_agent

see apps/codexgraph_agent/README.md

Related issue number

Checklist

  • The pull request title is a good summary of the changes - it will be used in the changelog
  • Unit tests for the changes exist
  • Run pre-commit install and pre-commit run --all-files before git commit, and passed lint check.
  • Some cases need DASHSCOPE_TOKEN_API to pass the Unit Tests, I have at least pass the Unit tests on local
  • Documentation reflects the changes where applicable
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

apps/codexgraph_agent/help.py Show resolved Hide resolved

return resp

def _run(self, cypher_queries, retries=5, **kwargs):
Copy link
Collaborator

Choose a reason for hiding this comment

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

后续,agent中核心方法 _run输入,输出要注明格式。如,这里的输入cypher_queries 是str, 输出是list。
其他agent也都加一下。
主要是之前其他的一些agent接入的时候用的非常wild,用raw prompt输入输出,返回也是str,或者yield str,
后续也会都改掉。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

Copy link
Contributor Author

@laptype laptype Jul 29, 2024

Choose a reason for hiding this comment

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

The output format of _run has been added.

def set_action_type_and_message(self):
pass

def _run(self, user_query, file_path: str = '', **kwargs):
Copy link
Collaborator

Choose a reason for hiding this comment

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

需要考虑yield 结果,支持streaming

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

@zzhangpurdue
Copy link
Collaborator

zzhangpurdue commented Jul 29, 2024

for core function, please add unit test case later.

@zzhangpurdue zzhangpurdue merged commit f6d20b7 into modelscope:master Jul 29, 2024
2 checks 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