Jiangzhou

Results 20 comments of Jiangzhou

@ambicuity thanks for taking this! assigned.

@Haleshot assigned. Thanks for taking this!

Thanks! Now an existing check failed. I think you may enable the `abi3` feature to the following line and have another try: https://github.com/cocoindex-io/cocoindex/blob/e3f47fda57ce47a84773d289b19ae9f32421f35f/.github/workflows/_test.yml#L57 Thanks!

Not really. This issue is still valid. It's about vector index method customization. Currently we only support [`HnswPq`](https://github.com/cocoindex-io/cocoindex/blob/536ebb0fef4519052cdb3e23cf61e255178ffc67/python/cocoindex/targets/lancedb.py#L418-L420)

@Haleshot Really fast! Looks very reasonable! Thanks!

About > **_PS_:** I took, what I believe is a pragmatic approach here - rather than replacing every `anyhow::Result` with `CResult` (which would've required touching hundreds of serde impls and...

> On the approach suggested above: I'll go about doing the following: > > * Rename `CError` → `Error`, `CResult` → `Result` in `cocoindex_utils` > * Update `prelude.rs` to use...

Created #1358. During test, noticed LanceDB's [LTS support](https://lancedb.com/docs/indexing/fts-index/) is for Cloud and Enterprise edition only. Nevertheless, we can checkin this feature first and waiting for users response to see if...

Hi @Haleshot, I don't have a strong preference. Sometimes it's easy to split but sometimes not. You can choose the way that makes your work easier. Even if you go...

Assigned to @harshitrwt in ordering. @Krishna-Sri-Charan thanks for volunteering. Welcome to take other issues!