-
Notifications
You must be signed in to change notification settings - Fork 15.8k
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
Standardize retriever integration docs #24908
Labels
🤖:docs
Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder
help wanted
Good issue for contributors
integration-docs
Ɑ: retriever
Related to retriever module
Comments
dosubot
bot
added
Ɑ: retriever
Related to retriever module
🤖:docs
Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder
labels
Jul 31, 2024
ccurme
added a commit
that referenced
this issue
Aug 2, 2024
Dear langchain maintainers, I add the wikipedia integration docs according to the [web docs](https://python.langchain.com/v0.2/docs/integrations/retrievers/wikipedia/), and follow the format of [tavily example](https://github.com/langchain-ai/langchain/blob/master/docs/docs/integrations/retrievers/tavily.ipynb) and [retriever template](https://github.com/langchain-ai/langchain/blob/master/libs/cli/langchain_cli/integration_template/docs/retrievers.ipynb), this is my first time contributing large repo. please let me know if I'm doing anything wrong, thank you! Topic related: #24908 --------- Co-authored-by: Chester Curme <chester.curme@gmail.com>
olgamurraft
pushed a commit
to olgamurraft/langchain
that referenced
this issue
Aug 16, 2024
Dear langchain maintainers, I add the wikipedia integration docs according to the [web docs](https://python.langchain.com/v0.2/docs/integrations/retrievers/wikipedia/), and follow the format of [tavily example](https://github.com/langchain-ai/langchain/blob/master/docs/docs/integrations/retrievers/tavily.ipynb) and [retriever template](https://github.com/langchain-ai/langchain/blob/master/libs/cli/langchain_cli/integration_template/docs/retrievers.ipynb), this is my first time contributing large repo. please let me know if I'm doing anything wrong, thank you! Topic related: langchain-ai#24908 --------- Co-authored-by: Chester Curme <chester.curme@gmail.com>
ccurme
added
investigate
Flagged for investigation.
integration-docs
and removed
investigate
Flagged for investigation.
labels
Aug 16, 2024
@ccurme Would like to contribute to this issue. |
9 tasks
cjumel
added a commit
to LinkupPlatform/langchain-linkup
that referenced
this issue
Nov 27, 2024
This is based on the following issues: langchain-ai/langchain#24908 langchain-ai/langchain#24820
cjumel
added a commit
to LinkupPlatform/langchain-linkup
that referenced
this issue
Nov 27, 2024
This is based on the following issues: langchain-ai/langchain#24908 langchain-ai/langchain#24820
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
🤖:docs
Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder
help wanted
Good issue for contributors
integration-docs
Ɑ: retriever
Related to retriever module
Privileged issue
Issue Content
Issue
To make our retriever integrations as easy to use as possible we need to make sure the docs for them are thorough and standardized. There are two parts to this: updating the retriever docstrings and updating the actual integration docs.
This needs to be done for each retriever integration, ideally with one PR per retriever.
Related to broader issues #21983 and #22005.
Docstrings
Each retriever class docstring should have the sections shown in the Appendix below. The sections should have input and output code blocks when relevant.
To build a preview of the API docs for the package you're working on run (from root of repo):
make api_docs_clean; make api_docs_quick_preview API_PKG=community
where
API_PKG=
should be the parent directory that houses the edited package (e.g. "community" forlangchain-community
).Doc pages
Each retriever docs page should follow this template.
See example here.
You can use the
langchain-cli
to quickly get started with a new integration docs page (run from root of repo):poetry run pip install -e libs/cli poetry run langchain-cli integration create-doc --name "foo-bar" --name-class FooBar --component-type Retriever --destination-dir ./docs/docs/integrations/retrievers/
where
--name
is the integration package name without the "langchain-" prefix and--name-class
is the class name without the "Retriever" postfix. This will create a template doc with some autopopulated fields at docs/docs/integrations/retrievers/foo_bar.ipynb.To build a preview of the docs you can run (from root):
make docs_clean make docs_build cd docs/build/output-new yarn yarn start
Appendix
Expected sections for the retriever class docstring.
See example here.
The text was updated successfully, but these errors were encountered: