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

[Feature]: Teach OpenDevin agents their name and email address #1439

Open
neubig opened this issue Apr 29, 2024 · 7 comments
Open

[Feature]: Teach OpenDevin agents their name and email address #1439

neubig opened this issue Apr 29, 2024 · 7 comments
Labels
enhancement New feature or request good first issue Good for newcomers severity:medium Problems that affect many users

Comments

@neubig
Copy link
Contributor

neubig commented Apr 29, 2024

What problem or use case are you trying to solve?

We recently added the ability to push to github: #1415

However, when doing so, our agents don't know their name and email address, so we should probably prompt them with a name and email address to use for PRs.

Do you have thoughts on the technical implementation?

This could be achieved by adding name and email to the config and defaults
, with defaults like OpenDevin and some email address we create.

This could then be referenced in the prompts provided to the agents, starting with MonologueAgent.

@neubig neubig added enhancement New feature or request good first issue Good for newcomers labels Apr 29, 2024
@ADTmux
Copy link

ADTmux commented Apr 30, 2024

Hey, I would like to work on this!

@neubig
Copy link
Contributor Author

neubig commented Apr 30, 2024

Great @ADTmux , please take a look!

@rbren rbren added the severity:low Minor issues, code cleanup, etc label May 2, 2024
@neubig
Copy link
Contributor Author

neubig commented May 16, 2024

Fixed by #1842

@neubig neubig closed this as completed May 16, 2024
@neubig
Copy link
Contributor Author

neubig commented May 19, 2024

Hmm, it seems like this is still not fixed...

I tried to do #1904 with OpenDevin, and it still doesn't have the appropriate username and email.

@neubig neubig reopened this May 19, 2024
@neubig neubig added severity:medium Problems that affect many users and removed severity:low Minor issues, code cleanup, etc labels May 19, 2024
@xingyaoww
Copy link
Collaborator

@neubig weird... Did the agent actually sets the git config by itself though?

@neubig
Copy link
Contributor Author

neubig commented May 19, 2024

Yes, because the git config is not set, the agent does it by itself. But this results in unpredictable behavior, where the name is different every time.

@xingyaoww
Copy link
Collaborator

@neubig I actually wonder what's the sandbox version you are using? I just tried with ghcr.io/opendevin/sandbox:main and it seems to work well for me: #1928

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers severity:medium Problems that affect many users
Projects
None yet
Development

No branches or pull requests

4 participants