-
Notifications
You must be signed in to change notification settings - Fork 56
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
GraphQL vector updates #1778
GraphQL vector updates #1778
Conversation
# Conflicts: # raphtory-graphql/Cargo.toml # raphtory-graphql/src/data.rs # raphtory-graphql/src/lib.rs # raphtory-graphql/src/model/graph/graph.rs # raphtory-graphql/src/model/graph/graphs.rs # raphtory-graphql/src/model/graph/mutable_graph.rs # raphtory-graphql/src/model/mod.rs # raphtory-graphql/src/server.rs # raphtory/src/serialise/serialise.rs # raphtory/src/vectors/template.rs # raphtory/src/vectors/vector_selection.rs # raphtory/src/vectors/vectorisable.rs
# Conflicts: # python/tests/graphql/edit_graph/test_deprecated_save.py # raphtory-graphql/src/data.rs # raphtory-graphql/src/lib.rs # raphtory-graphql/src/model/algorithms/global_search.rs # raphtory-graphql/src/model/algorithms/similarity_search.rs # raphtory-graphql/src/model/mod.rs # raphtory-graphql/src/model/plugins/query_plugin.rs # raphtory-graphql/src/model/plugins/vector_algorithm_plugin.rs # raphtory-graphql/src/python/server/server.rs # raphtory-graphql/src/server.rs
|
||
println!("Graph loaded = {}", folder.get_original_path_str()); | ||
Ok(Self { | ||
graph: IndexedGraph::from_graph(&graph)?, |
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.
We need to test how long the indexing is taking on big graphs
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.
Couple small changes and we are good to go!
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
What changes were proposed in this pull request?
Why are the changes needed?
Does this PR introduce any user-facing change? If yes is this documented?
How was this patch tested?
Issues
If this resolves any issues, please link to them here, the format is a KEYWORD followed by @_
KEYWORDS available are
close
,closes
,closed
,fix
,fixes
,fixed
,resolve
,resolves
,resolved
.Please delete this text before creating your PR
Are there any further changes required?