milvus-sdk-rust icon indicating copy to clipboard operation
milvus-sdk-rust copied to clipboard

[Bug]: metric_type COSINE try_from panic

Open franklucky001 opened this issue 9 months ago • 3 comments

Is there an existing issue for this?

  • [x] I have searched the existing issues

Describe the bug

pub enum MetricType {
    L2,
    IP,
    HAMMING,
    JACCARD,
    TANIMOTO,
    SUBSTRUCTURE,
    SUPERSTRUCTURE,
   // COSINE missing define here
}

代码中 metric_type enum中缺少COSINE的枚举定义

Expected Behavior

No response

Steps/Code To Reproduce behavior


Environment details

- Hardware/Softward conditions (OS, CPU, GPU, Memory):
- Method of installation (Docker, or from source):
- Milvus version (v0.3.1, or v0.4.0):
- Milvus configuration (Settings you made in `server_config.yaml`):

Anything else?

No response

franklucky001 avatar Feb 24 '25 08:02 franklucky001

what version are u using?

seems to be a version of last decade...

xiaofan-luan avatar Feb 24 '25 14:02 xiaofan-luan

@xiaofan-luan https://github.com/milvus-io/milvus-sdk-rust/blob/main/src/index/mod.rs#L46 main branch

franklucky001 avatar Feb 24 '25 14:02 franklucky001

gotcha.

I guess rust RDK is not maintained for a while.

For milvus version older than 2.3, cosine is not supported.

We need some volunteers to update it, and will GA rust SDK later this year

xiaofan-luan avatar Feb 24 '25 14:02 xiaofan-luan