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

[Enhancement] PR update message with summarized details #188

Closed
malhotra5 opened this issue Oct 25, 2024 · 11 comments · Fixed by #203
Closed

[Enhancement] PR update message with summarized details #188

malhotra5 opened this issue Oct 25, 2024 · 11 comments · Fixed by #203
Labels
fix-me Try to automatically fix the issue with OpenHands

Comments

@malhotra5
Copy link
Contributor

malhotra5 commented Oct 25, 2024

Current

Openhands resolver updates PR branches based on review comments. When the PR branch is updated with new code, openhands resolver creates a new comment thread containing

"New openhands update"

to indicate the PR has been updated.

Also, a reply is left for every unresolved review comment thread with a summary of code changes made to address that specific comment.

Enhancement

Change the PR update message "New openhands update" to be an AI generated summary of all replies made to each unresolved comment thread (as opposed to the fixed message).

An example of how to call a language model (which can be used to create the AI-generated summary) can be found in the guess_success() function in resolve_issues.py.

Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

The workflow to fix this issue encountered an error. Please check the workflow logs for more information.

Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

The workflow to fix this issue encountered an error. Please check the workflow logs for more information.

Copy link

OpenHands started fixing the issue! You can monitor the progress here.

1 similar comment
Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

An attempt was made to automatically fix this issue, but it was unsuccessful. A branch named 'openhands-fix-issue-188' has been created with the attempted changes. You can view the branch here. Manual intervention may be required.

@neubig neubig added fix-me Try to automatically fix the issue with OpenHands and removed fix-me-experimental labels Oct 28, 2024
Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

An attempt was made to automatically fix this issue, but it was unsuccessful. A branch named 'openhands-fix-issue-188-try2' has been created with the attempted changes. You can view the branch here. Manual intervention may be required.

@neubig neubig added fix-me Try to automatically fix the issue with OpenHands and removed fix-me Try to automatically fix the issue with OpenHands labels Oct 30, 2024
Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

An attempt was made to automatically fix this issue, but it was unsuccessful. A branch named 'openhands-fix-issue-188-try3' has been created with the attempted changes. You can view the branch here. Manual intervention may be required.

neubig added a commit that referenced this issue Oct 31, 2024
…ils (#203)

* Fix issue #188: '[Enhancement] PR update message with summarized details'

* Update

* Fix tests

---------

Co-authored-by: openhands <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix-me Try to automatically fix the issue with OpenHands
Projects
None yet
2 participants