-
Notifications
You must be signed in to change notification settings - Fork 244
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: add to_list and to_pandas api's #556
Conversation
e41223c
to
dad105b
Compare
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.
There's two more to_df
s here:
lancedb/docs/src/notebooks/multimodal_search.ipynb
Lines 182 to 185 in dad105b
" f\"duckdb.sql('{query}').to_df()\"\n", | |
" ) \n", | |
" diffusiondb = tbl.to_lance()\n", | |
" return (_extract(duckdb.sql(query).to_df()), code)\n", |
Looks good otherwise.
Edit: nevermind, that's duckdb
@@ -80,14 +80,14 @@ def handler(event, context): | |||
# Shape of SIFT is (128,1M), d=float32 | |||
query_vector = np.array(event['query_vector'], dtype=np.float32) | |||
|
|||
rs = table.search(query_vector).limit(2).to_df() | |||
rs = table.search(query_vector).limit(2).to_list() |
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.
What does rs
stand for?
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.
ResultSet
Add `to_list` to return query results as list of python dict (so we're not too pandas-centric). Closes lancedb#555 Add `to_pandas` API and add deprecation warning on `to_df`. Closes lancedb#545 Co-authored-by: Chang She <chang@lancedb.com>
add a missing boolean check
Add
to_list
to return query results as list of python dict (so we're not too pandas-centric). Closes #555Add
to_pandas
API and add deprecation warning onto_df
. Closes #545