-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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 inference readme #2969
update inference readme #2969
Conversation
❌ pre-commit failed. |
Hi. Could you also incorporate some of the instructions from #2595 into this? I think that PR is stale |
updated with the updates of the other PR @olliestanley |
❌ pre-commit failed. |
@Chuseuiti can you run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Fixed inference README with working version and deleted a lot of the extra variants information to simplify the document, as they are derivations of the same steps.
The motivation is that this container is not part of the docker compose, neither exist in the existing repository in the main branch:
inference-text-client