Skip to content

Conversation

@sunyuhan1998
Copy link
Contributor

As mentioned in issue #3374: RetrievalAugmentationAdvisor is part of spring-ai-rag.

… need to use the RetrievalAugmentationAdvisor, you should depend on spring-ai-rag.

Signed-off-by: Sun Yuhan <[email protected]>
@ilayaperumalg
Copy link
Member

@sunyuhan1998 Thanks for the PR fixing the documentation!

Spring AI provides out-of-the-box support for common RAG flows using the `Advisor` API.

To use the `QuestionAnswerAdvisor` or `RetrievalAugmentationAdvisor`, you need to add the `spring-ai-advisors-vector-store` dependency to your project:
To use the `QuestionAnswerAdvisor`, you need to add the `spring-ai-advisors-vector-store` dependency to your project:
Copy link
Member

Choose a reason for hiding this comment

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

We need to add VectorStoreChatMemoryAdvisor here. Will add it when merging.

Copy link
Contributor Author

@sunyuhan1998 sunyuhan1998 May 29, 2025

Choose a reason for hiding this comment

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

@ilayaperumalg Ok, no problem. By the way, I have previously pushed another PR which I think is also a bug, can you help to review this PR? #3268. thank you!

Copy link
Member

Choose a reason for hiding this comment

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

Sure, will check it out and thank you for the PRs!

@ilayaperumalg ilayaperumalg self-assigned this May 29, 2025
@ilayaperumalg ilayaperumalg added documentation Improvements or additions to documentation for: backport-to-1.0.x labels May 29, 2025
@ilayaperumalg ilayaperumalg added this to the 1.1.x milestone May 29, 2025
@ilayaperumalg
Copy link
Member

Updated the doc to include VectorStoreChatMemoryAdvisor and merged as a0d187d. Also, back-ported into 1.0.x as a0c76a0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation for: backport-to-1.0.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants