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.

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

build.sh switch to use RAPIDS magic value #3127

Conversation

robertmaynard
Copy link
Contributor

rapids-cmake 23.02 is deprecating the magic value of ALL since it doesn't cleanly map to the cmake magic value of all. Instead we use RAPIDS which better represents the architectures we are building for.

rapids-cmake 23.02 is deprecating the magic value of `ALL` since
it doesn't cleanly map to the cmake magic value of `all`. Instead
we use `RAPIDS` which better represents the architectures we are
building for.
@robertmaynard robertmaynard changed the title ./build.sh switch to use RAPIDS magic value build.sh switch to use RAPIDS magic value Jan 10, 2023
@BradReesWork BradReesWork requested review from vyasr, rlratzel and BradReesWork and removed request for vyasr and rlratzel January 10, 2023 20:57
@BradReesWork BradReesWork added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jan 10, 2023
@BradReesWork BradReesWork added this to the 23.02 milestone Jan 10, 2023
@BradReesWork
Copy link
Member

rerun tests

1 similar comment
@BradReesWork
Copy link
Member

rerun tests

@codecov-commenter
Copy link

codecov-commenter commented Jan 12, 2023

Codecov Report

Base: 57.60% // Head: 55.15% // Decreases project coverage by -2.44% ⚠️

Coverage data is based on head (7470d1f) compared to base (324f370).
Patch coverage: 39.60% of modified lines in pull request are covered.

Additional details and impacted files
@@               Coverage Diff                @@
##           branch-23.02    #3127      +/-   ##
================================================
- Coverage         57.60%   55.15%   -2.45%     
================================================
  Files               147      147              
  Lines              9227     9375     +148     
================================================
- Hits               5315     5171     -144     
- Misses             3912     4204     +292     
Impacted Files Coverage Δ
python/cugraph-pyg/cugraph_pyg/loader/dispatch.py 28.57% <ø> (ø)
...t/cugraph_service_client/cugraph_service_thrift.py 68.18% <ø> (ø)
...-service/server/cugraph_service_server/__main__.py 0.00% <0.00%> (ø)
...rvice_server/testing/benchmark_server_extension.py 0.00% <0.00%> (-40.39%) ⬇️
python/cugraph/cugraph/__init__.py 100.00% <ø> (ø)
...ugraph/cugraph/dask/structure/mg_property_graph.py 11.68% <0.00%> (-0.30%) ⬇️
python/cugraph/cugraph/sampling/node2vec.py 78.78% <ø> (ø)
python/cugraph/cugraph/structure/__init__.py 100.00% <ø> (ø)
python/cugraph/cugraph/structure/graph_classes.py 79.67% <ø> (+0.27%) ⬆️
python/cugraph/cugraph/testing/mg_utils.py 25.71% <0.00%> (-5.33%) ⬇️
... and 22 more

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.

@BradReesWork
Copy link
Member

/merge

@BradReesWork BradReesWork removed the request for review from vyasr January 12, 2023 21:26
@BradReesWork
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit a02e876 into rapidsai:branch-23.02 Jan 12, 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.

4 participants