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 inference readme #2969

Merged
merged 3 commits into from
Apr 29, 2023
Merged

update inference readme #2969

merged 3 commits into from
Apr 29, 2023

Conversation

Chuseuiti
Copy link
Contributor

@Chuseuiti Chuseuiti commented Apr 29, 2023

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

@github-actions
Copy link

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

@olliestanley
Copy link
Collaborator

Hi. Could you also incorporate some of the instructions from #2595 into this? I think that PR is stale

@olliestanley olliestanley linked an issue Apr 29, 2023 that may be closed by this pull request
@Chuseuiti
Copy link
Contributor Author

updated with the updates of the other PR @olliestanley

@github-actions
Copy link

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

@andrewm4894
Copy link
Collaborator

@Chuseuiti can you run pre-commit run --all-files and then add that commit, it looks like it just wants to clean some whitespace and end of file stuff.

Copy link
Collaborator

@olliestanley olliestanley left a comment

Choose a reason for hiding this comment

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

Thanks!

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.

Inference readme outdated
4 participants