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

Example for SWE agent #173

Merged
merged 46 commits into from
May 15, 2024
Merged

Conversation

garyzhang99
Copy link
Collaborator


name: Pull Request
about: Create a pull request

Description

[Partially implement SWE-agent as an example]

Checklist

Please check the following items before code is ready to be reviewed.

  • Code has passed all tests
  • Docstrings have been added/updated in Google Style
  • Documentation has been updated
  • Code is ready for review

@garyzhang99 garyzhang99 self-assigned this Apr 23, 2024
@garyzhang99 garyzhang99 added the enhancement New feature or request label Apr 23, 2024
@garyzhang99 garyzhang99 changed the title [WIP] Example for SWE agent Example for SWE agent May 8, 2024
Copy link
Collaborator

@DavdGao DavdGao left a comment

Choose a reason for hiding this comment

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

Please see inline comments.

examples/swe_agent/main.ipynb Outdated Show resolved Hide resolved
examples/swe_agent/swe_agent_prompts.py Show resolved Hide resolved
examples/swe_agent/swe_agent_prompts.py Show resolved Hide resolved
src/agentscope/service/file/check.py Outdated Show resolved Hide resolved
src/agentscope/service/file/common.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@DavdGao DavdGao left a comment

Choose a reason for hiding this comment

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

Please see inline comments

examples/swe_agent/main.ipynb Outdated Show resolved Hide resolved
examples/swe_agent/main.ipynb Outdated Show resolved Hide resolved
examples/swe_agent/main.ipynb Outdated Show resolved Hide resolved
examples/swe_agent/main.ipynb Outdated Show resolved Hide resolved
examples/swe_agent/main.ipynb Outdated Show resolved Hide resolved
examples/swe_agent/main.ipynb Outdated Show resolved Hide resolved
src/agentscope/service/file/common.py Outdated Show resolved Hide resolved
src/agentscope/service/file/common.py Show resolved Hide resolved
@DavdGao
Copy link
Collaborator

DavdGao commented May 15, 2024

Add this example in README.md and README_zh.md

Copy link
Collaborator

@DavdGao DavdGao left a comment

Choose a reason for hiding this comment

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

see inline comments, others lgtm

README.md Outdated Show resolved Hide resolved
README_ZH.md Outdated Show resolved Hide resolved
examples/swe_agent/main.ipynb Outdated Show resolved Hide resolved
examples/swe_agent/main.ipynb Outdated Show resolved Hide resolved
Copy link
Collaborator

@DavdGao DavdGao left a comment

Choose a reason for hiding this comment

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

LGTM

@DavdGao DavdGao merged commit 8a0cd56 into modelscope:main May 15, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants