Skip to content

define the interface for the "pyramid"#167

Merged
inabao merged 8 commits into
mainfrom
pyramid
Dec 18, 2024
Merged

define the interface for the "pyramid"#167
inabao merged 8 commits into
mainfrom
pyramid

Conversation

@inabao

@inabao inabao commented Nov 27, 2024

Copy link
Copy Markdown
Collaborator

Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
Comment thread examples/cpp/simple_pyramid.cpp
Comment thread src/dataset_impl.h
class DatasetImpl : public Dataset {
using var = std::variant<int64_t, const float*, const int8_t*, const int64_t*>;
using var =
std::variant<int64_t, const float*, const int8_t*, const int64_t*, const std::string*>;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please add the field comments?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The explanation of these fields can be found at the interface.

@inabao inabao self-assigned this Dec 3, 2024
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
Comment thread src/dataset_impl.h
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>

@jiaweizone jiaweizone left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread examples/cpp/simple_pyramid.cpp Outdated
Comment thread src/index/pyramid.h Outdated
Comment thread src/index/pyramid.h
Comment thread src/index/pyramid.h
Comment thread examples/cpp/simple_pyramid.cpp Outdated

std::mt19937 rng;
rng.seed(47);
std::uniform_real_distribution<> distrib_real;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add template float

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>

@LHT129 LHT129 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@inabao
inabao merged commit cf12139 into main Dec 18, 2024
@inabao
inabao deleted the pyramid branch December 18, 2024 02:57
Roxanne0321 pushed a commit to Roxanne0321/vsag that referenced this pull request Mar 3, 2025
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
jiacai2050 pushed a commit to jiacai2050/vsag that referenced this pull request Mar 6, 2025
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
LHT129 pushed a commit that referenced this pull request May 11, 2026
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
Signed-off-by: LHT129 <tianlan.lht@antgroup.com>
Sia-Sheerland pushed a commit to Sia-Sheerland/vsag that referenced this pull request Jun 26, 2026
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
Signed-off-by: Sia Sheerland <x1075956441x@163.com>

Signed-off-by: Sia Sheerland <x1075956441x@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants