All-Hands-AI / openhands-resolver

A system that tries to resolve all issues on a github repo with OpenHands.
MIT License
85 stars 20 forks source link

Update openhands-ai to 0.12.0 #218

Closed neubig closed 2 weeks ago

neubig commented 2 weeks ago

This PR updates openhands-ai to version 0.12.0 and fixes the runtime import path.

Changes:

All tests are passing.

Fixes https://github.com/All-Hands-AI/openhands-resolver/pull/218

malhotra5 commented 2 weeks ago

I believe await runtime.connect() also needs to be added after initialize_runtime(runtime)

this is around line 246 in openhands_resolver/resolve_issues.py

neubig commented 2 weeks ago

Thanks @malhotra5 , I conveyed this to OpenHands :)

malhotra5 commented 2 weeks ago

I believe await runtime.connect() also needs to be added after initialize_runtime(runtime)

this is around line 246 in openhands_resolver/resolve_issues.py

apologies there is a correction, await runtime.connect() needs to be added BEFORE initialize_runtime(runtime)

neubig commented 2 weeks ago

Hey @malhotra5 , hmm, I tried debugging this now and am having issues running it locally. I ran out of time, but if you have a minute it'd be great if you could take a look!

Namely I'm getting something that looks like this:


==============
[Agent Controller default] LEVEL 0 LOCAL STEP 0 GLOBAL STEP 0

==============
[Agent Controller default] LEVEL 0 LOCAL STEP 1 GLOBAL STEP 1

==============
[Agent Controller default] LEVEL 0 LOCAL STEP 2 GLOBAL STEP 2

==============
[Agent Controller default] LEVEL 0 LOCAL STEP 3 GLOBAL STEP 3
xingyaoww commented 2 weeks ago

I think this is expected: https://github.com/All-Hands-AI/OpenHands/pull/4563 removed the logging for all action/obs here by default -- We now need to set DEBUG=true to display everything i think?

neubig commented 2 weeks ago

Awesome, thanks @xingyaoww . @malhotra5 please disregard.