Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Migrate old graph to legacy directory/namespace #1675

Merged
merged 4 commits into from
Jun 21, 2021

Conversation

ChuckHastings
Copy link
Collaborator

Code cleanup. We want to migrate the new graph object into the cugraph namespace. Moving the legacy graph objects into a legacy namespace to clear the way.

@ChuckHastings ChuckHastings requested a review from a team as a code owner June 17, 2021 21:54
@ChuckHastings ChuckHastings self-assigned this Jun 17, 2021
@ChuckHastings ChuckHastings added 2 - In Progress breaking Breaking change improvement Improvement / enhancement to an existing function labels Jun 17, 2021
@ChuckHastings ChuckHastings added this to the 21.08 milestone Jun 17, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #1675 (164cfc7) into branch-21.08 (1030a49) will decrease coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##           branch-21.08    #1675      +/-   ##
================================================
- Coverage         59.77%   59.66%   -0.12%     
================================================
  Files                80       80              
  Lines              3540     3540              
================================================
- Hits               2116     2112       -4     
- Misses             1424     1428       +4     
Impacted Files Coverage Δ
python/cugraph/structure/number_map.py 63.92% <0.00%> (-1.18%) ⬇️
python/cugraph/utilities/utils.py 72.34% <0.00%> (-0.71%) ⬇️

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 1030a49...164cfc7. Read the comment docs.

@BradReesWork
Copy link
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit d01da40 into rapidsai:branch-21.08 Jun 21, 2021
@ChuckHastings ChuckHastings deleted the fea_move_legacy_graph branch July 29, 2021 15:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
breaking Breaking change improvement Improvement / enhancement to an existing function
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants