-
Notifications
You must be signed in to change notification settings - Fork 128
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(builder): add embedding model #385
Conversation
andylau-55
commented
Nov 1, 2024
- Provide built-in embedding model BAAI/bge-base-zh-v1.5.
- Implement getting vector dimensions of the embedding model automatically.
…nt getting vector dimensions of the embedding model automatically
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.
license-eye has checked 1544 files.
Valid | Invalid | Ignored | Fixed |
---|---|---|---|
1516 | 2 | 26 | 0 |
Click to see the invalid file list
- python/knext/init.rc
- python/knext/knext/reasoner/rest/models/edge.py
Use this command to fix any missing license headers
```bash
docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix
</details>
@@ -1,15 +1,4 @@ | |||
# coding: utf-8 |
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.
# coding: utf-8 | |
# Copyright 2023 OpenSPG Authors | |
# | |
# Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except | |
# in compliance with the License. You may obtain a copy of the License at | |
# | |
# http://www.apache.org/licenses/LICENSE-2.0 | |
# | |
# Unless required by applicable law or agreed to in writing, software distributed under the License | |
# is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express | |
# or implied. | |
# coding: utf-8 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #385 +/- ##
=========================================
Coverage ? 49.03%
Complexity ? 314
=========================================
Files ? 718
Lines ? 24987
Branches ? 4101
=========================================
Hits ? 12253
Misses ? 11314
Partials ? 1420 ☔ View full report in Codecov by Sentry. |
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.
LGTM
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.
LGTM