-
Notifications
You must be signed in to change notification settings - Fork 343
Updated create_index #1571
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
Updated create_index #1571
Conversation
- `"ip"`: Inner product. | ||
- `"l2"`: Euclidean distance. | ||
- `"cosine"`: Cosine similarity. | ||
- Parameter settings for a secondary index: |
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.
No parameter.
- `"pq_subspace_num"`: *Required* - Recommended: "32". | ||
- `"pq_subspace_bits"`: *Required* - Recommended: "8". | ||
- Parameter settings for a full text index: | ||
- `"ANALYZER"`: *Optional* - Defaults to `"standard"` |
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.
Other values?
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.
All parameters should be case-insensitive.
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.
These parameters are case-sensitive. @JinHai-CN
The name of the column to build index on. Must not be empty. | ||
- **index_type**: `IndexType`, *Required* | ||
Index type. You may want to import `infinity.index` to set `IndexType`: `from infinity.index import IndexType` | ||
- `Hnsw`: A HNSW index. |
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.
HNSW
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.
Hnsw
- **index_param_list**: `list[InitParameter(str, str)]` | ||
A list of `InitParameter` objects specifying parameter settings for the chosen index type. Each object handles one parameter setting. To set a specific index parameter, pass the parameter name and its corresponding value as two separate strings to the `InitParameter` object: | ||
- Parameter settings for an HNSW index: | ||
- `"M"`: *Optional* - Defaults to`"16"`. |
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.
Case insensitive
|
||
### Returns | ||
|
||
- Success: An index object. |
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.
Return value, should have error message
What problem does this PR solve?
Issue link:#1490
Type of change