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

Add api-inference docs #522

Closed
wants to merge 1 commit into from
Closed

Add api-inference docs #522

wants to merge 1 commit into from

Conversation

mishig25
Copy link
Collaborator

@mishig25 mishig25 commented Nov 22, 2022

Closes #520

@mishig25 mishig25 marked this pull request as draft November 22, 2022 10:47
@merveenoyan
Copy link
Contributor

This PR is doing God's work. Thanks @mishig25 🤗 This will make adding docs much easier!
(I rarely add things to api-inference docs and I forget about dev setup every time 😂 for me, it didn't make any sense that it was outside of this repo)

@mishig25
Copy link
Collaborator Author

Closing this issue for now. The detailed parameters page (probably the most important page of the doc) gets auto generated from api-inference. Specifically, all the code snippets (see attached screenshot) are part of python test files, which gets used in generating the doc. Therefore, it is not possible to open-source docs only without open-sourcing the test files.

image

cc: @michaelbenayoun

@mishig25 mishig25 closed this Nov 23, 2023
@julien-c julien-c deleted the api_inference_docs branch November 23, 2023 10:57
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.

Add Inference API docs under this repo
2 participants