OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

Fix issue #6060: Speed up loading of GitHub repositories in the front-end hero page

Open neubig opened this issue 1 year ago • 3 comments

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

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

Speed up loading of GitHub repositories in the front-end hero page


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

This PR increases the pagination of repo requests in the frontend to download 1000 entries, which will speed up loading.

Note that this pulls in the removal of open-cv from #6064, which was necessary to get it to run on my machine.


Link of any specific issues this addresses

Fixes #6060


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:e50020d-nikolaik   --name openhands-app-e50020d   docker.all-hands.dev/all-hands-ai/openhands:e50020d

neubig avatar Jan 06 '25 03:01 neubig

@openhands-agent This PR is failing frontend unit tests and linting. Please examine the github workflows for linting/testing the frontend, run the workflows, and fix any errors.

neubig avatar Jan 06 '25 03:01 neubig

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

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

Status: UNRESOLVED ❌

Summary: The changes made were insufficient to address the original issues. The agent only mentioned editing a test file without providing:

  • Details of specific test failures
  • Documentation of fixes made
  • Confirmation of linting issues being addressed
  • Verification that tests/linting now pass

Required Actions:

  1. Review GitHub workflow logs for specific failures
  2. Implement and document necessary fixes
  3. Verify fixes locally
  4. Confirm passing CI checks
  5. Provide evidence of resolution

Recommendation: A complete resolution requires thorough documentation of the issues found, changes made, and verification of success through passing CI checks. The current changes do not meet these requirements.

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

Thanks @amanape, I thought I read somewhere that the maximum was 1000... Sorry to waste your time on this.

neubig avatar Jan 10 '25 09:01 neubig