OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

Fix issue #6046: Document GPU usage

Open openhands-agent opened this issue 1 year ago • 6 comments

This pull request fixes #6046.

The issue has been successfully resolved. The AI agent has properly documented the GPU support feature that was added in PR #6042 by:

  1. Adding the documentation in an appropriate location (under "Container Image" section, which makes sense for hardware configuration)
  2. Including all necessary details about the enable_gpu configuration option:
    • The parameter type (boolean)
    • Default value (false)
    • Purpose (enabling GPU support in sandbox containers)
    • Scope (access to all available GPUs)
  3. Placing it in the advanced configuration section as requested in the original issue

The documentation addition meets all the requirements from the original issue: it documents the GPU feature, places it in an appropriate location for advanced configuration options, and provides clear, complete information for users who need to use GPU support in their OpenHands environment.

This can be considered complete and ready for human review.

Automatic fix generated by OpenHands 🙌


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:84af1fa-nikolaik   --name openhands-app-84af1fa   docker.all-hands.dev/all-hands-ai/openhands:84af1fa

openhands-agent avatar Jan 05 '25 06:01 openhands-agent

Has this been tested by anyone?

li-boxuan avatar Jan 05 '25 06:01 li-boxuan

@li-boxuan : it has been reported as working https://github.com/All-Hands-AI/OpenHands/issues/4230#issuecomment-2567637647

I agree that one of the maintainers should probably test it as well.

neubig avatar Jan 05 '25 07:01 neubig

Amped to see this being included into OpenHands

BradKML avatar Jan 13 '25 17:01 BradKML

@neubig just wondering, how often does a PR need to update based on the changes in the main branch?

BradKML avatar Jan 14 '25 02:01 BradKML

@BradKML you don't need to keep it up to date that often with main. Unless there's a high chance of conflicts in which it's in your best interest to do so.

mamoodi avatar Jan 14 '25 22:01 mamoodi

This PR is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

github-actions[bot] avatar Feb 14 '25 01:02 github-actions[bot]

This PR was closed because it has been stalled for over 30 days with no activity.

github-actions[bot] avatar Feb 21 '25 01:02 github-actions[bot]