unum-cloud / usearch

Fast Open-Source Search & Clustering engine × for Vectors & 🔜 Strings × in C++, C, Python, JavaScript, Rust, Java, Objective-C, Swift, C#, GoLang, and Wolfram 🔍
https://unum-cloud.github.io/usearch/
Apache License 2.0
2.15k stars 130 forks source link

Bug: Quantization enum not public in Golang SDK #326

Closed comunidadio closed 8 months ago

comunidadio commented 9 months ago

Describe the bug

Quantization enums (f32, f16, f64, i8, b1) aren't accessible to users of Go SDK since Go requires capitalized public exported fields/methods (ie. should be F32, F16, ...)

https://github.com/unum-cloud/usearch/blob/c2463ca3a043c26e99f50cfbf31232a8a646d060/golang/lib.go#L59C1-L60C1

Steps to reproduce

Can't use SDK's Quantization enum in IndexConfig as they are not exported.

Expected behavior

Quantization enum values available to Go SDK users.

USearch version

latest

Operating System

macos

Hardware architecture

Arm

Which interface are you using?

Other bindings

Contact Details

No response

Is there an existing issue for this?

Code of Conduct

ashvardanian commented 9 months ago

You are right, @comunidadio! Would you be able to open a PR?

comunidadio commented 9 months ago

@ashvardanian Sure, I can do that sometime this week. May try add other missing features in the Go binding, time permitting.

ashvardanian commented 8 months ago

It was solved by @laitassou in the latest release. @comunidadio let us know if it works and feel free to share more suggestions and patches for the Go SDK 🤗