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

Updates the Spring AI dependency to version 1.0.0-M5. #325

Merged
merged 5 commits into from
Dec 30, 2024

Conversation

fuyou-lxm
Copy link
Contributor

Describe what this PR does / why we need it

Updates the Spring AI dependency to version 1.0.0-M5.

Does this pull request fix one issue?

none

Describe how you did it

  1. Replaced the FunctionCallbackContext parameter with FunctionCallbackResolver in the dashscope model constructors. (#1804)
  2. Adapted the AdvisedRequest builder method changes and modified the relevant code.
  3. Adapted the changes in the Document structure and modified the relevant code in AnalyticdbVector. (#1883)

Describe how to verify it

none

Special notes for reviews

  1. Due to changes in the dashscope and AnalyticdbVector related code, a review from the relevant authors is needed.
  2. I added TODO comments in the dashscope related code to highlight the points that need review

…ng AI

- Update Spring AI dependency to version 1.0.0-M5
- Adjust code to adapt to the new Document and SearchRequest classes
- Modify relevant methods to use the new API
- Optimize part of the code structure to improve maintainability
- Formatted code in multiple classes, adjusted indentation and line breaks
- Cleaned up some redundant comments
…atModel classes—these comments referenced structural changes and adaptation issues that needed review, but there were no actual code changes.
@chickenlj chickenlj merged commit 31c52cd into alibaba:main Dec 30, 2024
1 of 2 checks passed
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.

2 participants