-
Notifications
You must be signed in to change notification settings - Fork 216
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
feat: expose use_index in LanceDB OSS #560
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Lance Release <lance-dev@lancedb.com> Co-authored-by: Rob Meng <rob.xu.meng@gmail.com> Co-authored-by: Will Jones <willjones127@gmail.com> Co-authored-by: Chang She <759245+changhiskhan@users.noreply.github.com> Co-authored-by: rmeng <rob@lancedb.com> Co-authored-by: Chang She <chang@lancedb.com> Co-authored-by: Rok Mihevc <rok@mihevc.org>
If you run the README javascript example in typescript, it complains that the type of limit is a function and cannot be set to a number.
A little verbose, but better than being non-discoverable ![Screenshot from 2023-10-11 16-26-02](https://github.com/lancedb/lancedb/assets/15766192/9ba539a7-0cf8-4d9e-94e7-ce5d37c35df0)
"append", which inserts new rows, and "overwrite", which replaces | ||
the entire content of the table with the new rows. |
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.
Drive-by change based on user feedback in discord. In retrospect, Table.add(mode="overwrite")
is a little confusing, since add
implies append. Perhaps we should make a separate Table.overwrite()
method?
It appears in Node that Cloud will just ignore the lancedb/node/src/remote/index.ts Lines 96 to 104 in 683824f
Maybe I should do that in Python too? |
7d3e6c9
to
99f5782
Compare
99f5782
to
61de91f
Compare
fix missing symbol on linux
Closes #535