- 
                Notifications
    
You must be signed in to change notification settings  - Fork 2k
 
fix: ZhiPuAiEmbedding supports custom dimensions #3668
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
fix: ZhiPuAiEmbedding supports custom dimensions #3668
Conversation
Signed-off-by: YunKui Lu <[email protected]>
| 
           @mxsl-gr Could you help reviewing this as well? Thank you  | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me and unit tests passed, feel free to merge
          
 Hi, @ilayaperumalg this PR looks good to me and unit tests passed, feel free to merge @YunKuiLu thanks for the PR  | 
    
| 
           @mxsl-gr @sunyuhan1998 Thanks for the review!  | 
    
| 
           @YunKuiLu Thanks for the PR!  | 
    
Fixes #3668 Signed-off-by: YunKui Lu <[email protected]> (cherry picked from commit 53f8f17)
Auto-cherry-pick to 1.0.x Fixes spring-projects#3668 Signed-off-by: YunKui Lu <[email protected]>
Auto-cherry-pick to 1.0.x Fixes spring-projects#3668 Signed-off-by: YunKui Lu <[email protected]>
I've run the integration tests locally, and everything checks out.
Closes #1857