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

[fix eval] Fix issues with miniwob remote runtime evaluation #5001

Merged
merged 9 commits into from
Nov 14, 2024

Conversation

ketan1741
Copy link
Contributor

End-user friendly description of the problem this fixes or functionality that this introduces

  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Give a summary of what the PR does, explaining any non-trivial design decisions

  • Fixes minor remote runtime setup issues with miniwob

Link of any specific issues this addresses

@xingyaoww xingyaoww enabled auto-merge (squash) November 14, 2024 17:36
@xingyaoww xingyaoww merged commit 852c90f into All-Hands-AI:main Nov 14, 2024
12 checks passed
@mamoodi
Copy link
Collaborator

mamoodi commented Nov 14, 2024

@ketan1741 the merge on this failed with Linting Python. Are you able to take a look and see why please?

@ketan1741
Copy link
Contributor Author

@ketan1741 the merge on this failed with Linting Python. Are you able to take a look and see why please?

Hey! @mamoodi There were linting issues but I fixed it and this seems to be successfully merged and passes all the tests. Is there anything I need to do?

@mamoodi
Copy link
Collaborator

mamoodi commented Nov 14, 2024

Nope all good it's fixed now thanks @ketan1741 !

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.

3 participants