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

Update openhands-ai to 0.12.0 #218

Merged
merged 10 commits into from
Oct 31, 2024
Merged

Update openhands-ai to 0.12.0 #218

merged 10 commits into from
Oct 31, 2024

Conversation

neubig
Copy link
Contributor

@neubig neubig commented Oct 31, 2024

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

Changes:

  • Update openhands-ai dependency to ^0.12.0
  • Fix runtime import from openhands.runtime.runtime to openhands.runtime.base
  • Update poetry.lock file

All tests are passing.

Fixes #218

@neubig neubig mentioned this pull request Oct 31, 2024
@malhotra5
Copy link
Contributor

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
Copy link
Contributor Author

neubig commented Oct 31, 2024

Thanks @malhotra5 , I conveyed this to OpenHands :)

@malhotra5
Copy link
Contributor

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
Copy link
Contributor Author

neubig commented Oct 31, 2024

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
Copy link

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

@neubig
Copy link
Contributor Author

neubig commented Oct 31, 2024

Awesome, thanks @xingyaoww . @malhotra5 please disregard.

@neubig neubig merged commit b91a2a3 into main Oct 31, 2024
2 checks passed
@neubig neubig deleted the update-openhands-ai-0.12.0 branch October 31, 2024 21:09
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.

4 participants