Skip to content

Promote triangle count from experimental#2671

Merged
rapids-bot[bot] merged 32 commits intorapidsai:branch-22.10from
jnke2016:branch-22.10_promote-triangle-count
Sep 19, 2022
Merged

Promote triangle count from experimental#2671
rapids-bot[bot] merged 32 commits intorapidsai:branch-22.10from
jnke2016:branch-22.10_promote-triangle-count

Conversation

@jnke2016
Copy link
Contributor

@jnke2016 jnke2016 commented Sep 8, 2022

In addition to replacing the current stable triangle_count API with experimental.triangle_count and updating the docs and test, this PR also adds support for the parameter start_list for both the SG and the MG implementation. It also renames triangle_count to triangles to be consistent with the previous API

closes #2667

@jnke2016 jnke2016 requested review from a team as code owners September 8, 2022 14:24
@jnke2016 jnke2016 requested a review from a team as a code owner September 8, 2022 17:59
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@codecov-commenter
Copy link

codecov-commenter commented Sep 8, 2022

Codecov Report

❗ No coverage uploaded for pull request base (branch-22.10@dfc640f). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@               Coverage Diff               @@
##             branch-22.10    #2671   +/-   ##
===============================================
  Coverage                ?   60.16%           
===============================================
  Files                   ?      111           
  Lines                   ?     6123           
  Branches                ?        0           
===============================================
  Hits                    ?     3684           
  Misses                  ?     2439           
  Partials                ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

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.

Looks good, I may have more suggestions later today but for now just some minor things.

@BradReesWork
Copy link
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 13f5d43 into rapidsai:branch-22.10 Sep 19, 2022
@jnke2016 jnke2016 deleted the branch-22.10_promote-triangle-count branch September 24, 2022 23:08
nickjcroucher added a commit to bacpop/PopPUNK that referenced this pull request Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Promote experimental.triangle_count to stable

4 participants