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 README file #472

Conversation

dorbanianas
Copy link
Collaborator

After adding the readme now we should update the getting started section so there's no need for the whole config that we used to do before, also I made an update to the whole structure to align with our current state

Copy link
Contributor

@neubig neubig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was successfully able to run make build and make setup-config but when I ran make run I got the following error, is this a known bug or should I create an issue?

Screenshot 2024-04-01 at 8 21 12 PM

Also, this PR has conflicts with main in README.md, could you resolve those?

@dorbanianas
Copy link
Collaborator Author

@neubig is there any errors shown in the log file or the terminal for the backend?

@neubig
Copy link
Contributor

neubig commented Apr 2, 2024

Ah, yes, lots of thes:

8:19:29 PM [vite] ws proxy error:
AggregateError [ECONNREFUSED]: 
    at internalConnectMultiple (node:net:1116:18)
    at afterConnectMultiple (node:net:1680:7)
8:19:29 PM [vite] http proxy error: /litellm-models
AggregateError [ECONNREFUSED]: 
    at internalConnectMultiple (node:net:1116:18)
    at afterConnectMultiple (node:net:1680:7)
8:19:29 PM [vite] http proxy error: /litellm-agents
AggregateError [ECONNREFUSED]: 
    at internalConnectMultiple (node:net:1116:18)
    at afterConnectMultiple (node:net:1680:7)
8:19:29 PM [vite] http proxy error: /litellm-models
AggregateError [ECONNREFUSED]: 
    at internalConnectMultiple (node:net:1116:18)
    at afterConnectMultiple (node:net:1680:7)
    ```

@neubig
Copy link
Contributor

neubig commented Apr 2, 2024

It's probably an instance of this: #508

@neubig neubig mentioned this pull request Apr 2, 2024
@dorbanianas dorbanianas force-pushed the ad/update-readme-file-after-adding-makefile branch from 5913b0d to fccaf44 Compare April 2, 2024 01:11
Copy link
Contributor

@neubig neubig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@neubig neubig merged commit fa40d37 into All-Hands-AI:main Apr 2, 2024
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.

2 participants