-
Notifications
You must be signed in to change notification settings - Fork 304
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
[WIP] cuGraph Documentation Example Errors #1862
Comments
Most if not all the errors mentioned are docstring mistakes which we are addressing in a PR. Below I provided a description on how to fix those errors before a PR is out Example 1:
This is due to a non existing directory. we currently do not have a bipartite.csv dataset in our repo but this will be added in the PR. Example 2:
This can be corrected like this. Please refer to https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.read_csv.html df = pandas.read_csv('./datasets/karate.csv', delimiter=' ', Example 3: Example 4: Example 5: you should first create a cugraph Di/Graph and then call 'get_two_hop_neighbors' like this Example 6: Example 7 |
Update cuGraph documentation and examples closes #1862 Authors: - Joseph Nke (https://github.com/jnke2016) Approvers: - Brad Rees (https://github.com/BradReesWork) - Rick Ratzel (https://github.com/rlratzel) URL: #1866
Describe the bug
cuGraph documentation has errors in some of the examples present. There are also some functionality that are not represented in the documentation. There are examples that relies on code that cannot be ran without errors in the most recent RAPIDS releases. I've began testing going back to 0.19 and there may have been some change to the API before that release. Not sure if these changes were intentional or if it was an uncaught bug.
Steps/Code to reproduce bug
Steps to reproduce the behavior:
1.Go to Rapids cuGraph documentation website
2.Click on the cuGraph API Reference link
3.Run the content for the examples therein as illustrated below
Expected behavior
There will be several examples that will create an error. Many examples miss details that could aide in implementation. The code will be a few commits behind from the 21.08. repo.
Environment details (please complete the following information):
Additional context
Examples of Discrepancies:
Example # 1
Error thrown below:
Where I am not able to test without a testing .csv to determine success
A second instance of the same issue
Where I am not able to test without a testing .csv to determine success
Error thrown below:
Example # 2
Where the .from_pandas_edgelist currently expects 2- or 3-tuples but seemingly used to allow a int32 parameter.
Error thrown below:
Example # 3
Error thrown below:
After adding .values received the below error:
Example # 4
Where I am not able to test without a input_data_path to determine success
Error thrown below:
A second instance of the same issue
Error thrown below:
A third instance of the same issue
Error thrown below:
A fourth instance of the same issue
Error thrown below:
A fifth instance of the same issue
Error thrown below:
Example # 5
Where .symmetrize currently expects . get_two_hop_neighbors to work in the above syntax amoungst other fixes where it seemingly is not available
Error thrown below:
Example # 6
Where cugraph.jaccard currently produces error
Error thrown below:
A second instance of the same issue
Error thrown below:
Example # 7
There is also opportunity to include additional examples for functionality not represented on this page. Afew examples of what could be missing are below
Desired outcome
Examples present in cuGraph documentation should be ready to replicated and implemented with less effort. Examples should reflect the commits made to the repositories during each release cycle. cuGraph functions and models work as expected.
Request impacts
Our cuGraph documentation is public and requires accurate information - Medium Priority
@BradReesWork @taureandyernv for awareness
The text was updated successfully, but these errors were encountered: