Move GaaS sources, tests, docs, scripts from the rapidsai/GaaS repo to the cugraph repo #2661
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…for graph management and a sampling algo (node2vec). This also includes test cases and a demo script.
Create initial version of GaaS client, server, tests, and demo
…tion extensions functions to Thrift and client API.
…xtension dir at startup.
…d tests to use that to determine if the server was ready, updated tests to cover preloaded extensions from the server CLI.
… end-to-end test and not just a unit test for the client API, made changes to allow tests to run without setting PYTHONPATH.
…owing thriftpy2.transport.TTransportError to be raised so users do not need to import thriftpy2 to catch exceptions from GaaS.
…with a default of 90 seconds.
…eeds cleanup and docs.
…removed debugging code.
…pport for retrieving the entire table if necessary instead of just individual rows.
…dgl_1 Add ability to load graphs server-side using custom functions
… tests to match the corresponding get_graph_vertex_dataframe_*() APIs. (this uncovered assumptions in the tests about the internal dataframe ordering which are currently non-deterministic, still working on this)
… bytes over RPC call.
… cudf to numpy (and omit tolist()) for return types. Still need to remove debug prints!
…' into branch-22.10-mg_updates
Co-authored-by: Rick Ratzel <[email protected]>
…mpling Updates to GaaS for Uniform Neighbor Sampling
…e optional, updated adn added tests accordingly.
…_*_data(). Added MG and SG tests to cover different ID types (exposed after a typo went unchecked). Added MG and SG tests to cover get_graph_*_data(). Minor cleanup and refactorings.
…ts, added FIXMEs.
…ugraph' into branch-22.10-gaas_migration
rlratzel
added
improvement
BradReesWork
approved these changes
Sep 2, 2022
|
@gpucibot merge |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #2652
Note to reviewers:
This is the initial PR which only moves source files and changes names in sources, docs, directories, etc. The unit tests were run (SG and MG) after the name change and they all pass.
One or more followup PRs will be open to add appropriate support for CI (ie. no MG tests run by default, limit to handler tests to simplify automating server subprocesses if necessary, etc.).
The entire GaaS repo is now under
python/cugraph_serviceas a peer tocugraphandpylibcugraph.