-
Notifications
You must be signed in to change notification settings - Fork 245
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
Notebook for new blog post on versioning #311
Conversation
@@ -0,0 +1,8 @@ | |||
// |
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.
No function declartion in this file?
Also, could you change header, and use #pragma once
instead of macro to protect header file?
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.
shoot, that's actually the next blog post. let me remove this
namespace lance::duckdb { | ||
|
||
template <typename T> | ||
void L2Distance(::duckdb::DataChunk &args, |
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 is the difference of this to https://github.com/eto-ai/lance/blob/main/integration/duckdb/src/lance/duckdb/vector_functions.cc#L25
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.
accident
22acab7
to
be2fac9
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.
LGTM
includes some minor refactoring to dataset conversion API