Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Docs for RMAT #1735

Merged
merged 4 commits into from
Jul 29, 2021
Merged

Docs for RMAT #1735

merged 4 commits into from
Jul 29, 2021

Conversation

BradReesWork
Copy link
Member

Fixed indentations so that docs appear correctly

@BradReesWork BradReesWork requested review from a team as code owners July 27, 2021 13:33
@BradReesWork BradReesWork added 3 - Ready for Review doc Documentation non-breaking Non-breaking change labels Jul 27, 2021
@BradReesWork BradReesWork added this to the 21.08 milestone Jul 27, 2021
@BradReesWork BradReesWork linked an issue Jul 27, 2021 that may be closed by this pull request
[ENH] add API documentation in RMAT code #1515
Closed
@codecov-commenter
Copy link

codecov-commenter commented Jul 27, 2021 •
edited
Loading

Codecov Report

Merging #1735 (804179e) into branch-21.08 (e5b3599) will increase coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 804179e differs from pull request most recent head 69a7ead. Consider uploading reports for the commit 69a7ead to get more accurate results
Impacted file tree graph

@@               Coverage Diff                @@
##           branch-21.08    #1735      +/-   ##
================================================
+ Coverage         59.72%   59.75%   +0.02%     
================================================
  Files                77       77              
  Lines              3521     3523       +2     
================================================
+ Hits               2103     2105       +2     
  Misses             1418     1418
Impacted Files Coverage Δ
python/cugraph/generators/rmat.py 34.44% <ø> (ø)
python/cugraph/_version.py 44.80% <0.00%> (+0.39%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e5b3599...69a7ead. Read the comment docs.

rlratzel
rlratzel requested changes Jul 28, 2021
View reviewed changes
Copy link
Contributor

@rlratzel rlratzel left a comment

Choose a reason for hiding this comment

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

Just a few typos.

python/cugraph/generators/rmat.py Outdated Show resolved Hide resolved
python/cugraph/generators/rmat.py Outdated Show resolved Hide resolved
python/cugraph/generators/rmat.py Outdated Show resolved Hide resolved
python/cugraph/generators/rmat.py Outdated Show resolved Hide resolved
jnke2016
jnke2016 reviewed Jul 28, 2021
View reviewed changes
Copy link
Contributor

@jnke2016 jnke2016 left a comment

Choose a reason for hiding this comment

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

Nothing to add other than the few typo mentioned by Rick

@BradReesWork
Copy link
Member Author

@gpucibot merge

1 similar comment
@BradReesWork
Copy link
Member Author

@gpucibot merge

@rapids-bot rapids-bot bot merged commit aab2a5c into rapidsai:branch-21.08 Jul 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jnke2016 jnke2016 jnke2016 left review comments

@rlratzel rlratzel rlratzel approved these changes

Assignees
No one assigned
Labels
doc Documentation non-breaking Non-breaking change
Projects
None yet
Milestone
21.08
Development

Successfully merging this pull request may close these issues.

[ENH] add API documentation in RMAT code
4 participants
@BradReesWork @codecov-commenter @rlratzel @jnke2016