Skip to content

[REVIEW] Enhance compile time#1267

Merged
BradReesWork merged 17 commits intorapidsai:branch-0.17from
aschaffer:enh_ext_compile_time_017
Nov 23, 2020
Merged

[REVIEW] Enhance compile time#1267
BradReesWork merged 17 commits intorapidsai:branch-0.17from
aschaffer:enh_ext_compile_time_017

Conversation

@aschaffer
Copy link
Copy Markdown
Collaborator

Look into alternative ways to instantiate templates to improve compilation time.

@aschaffer aschaffer requested a review from a team as a code owner November 12, 2020 00:02
@GPUtester
Copy link
Copy Markdown
Contributor

Please update the changelog in order to start CI tests.

View the gpuCI docs here.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 12, 2020

Codecov Report

Merging #1267 (e283aa2) into branch-0.17 (1be056b) will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           branch-0.17    #1267      +/-   ##
===============================================
+ Coverage        57.84%   57.92%   +0.08%     
===============================================
  Files               63       65       +2     
  Lines             2780     2800      +20     
===============================================
+ Hits              1608     1622      +14     
- Misses            1172     1178       +6     
Impacted Files Coverage Δ
python/cugraph/layout/force_atlas2.py 52.94% <0.00%> (-7.06%) ⬇️
python/cugraph/cores/k_core.py 85.71% <0.00%> (-1.79%) ⬇️
python/cugraph/__init__.py 100.00% <0.00%> (ø)
python/cugraph/dask/link_analysis/pagerank.py 25.80% <0.00%> (ø)
python/cugraph/linear_assignment/lap.py 77.77% <0.00%> (ø)
python/cugraph/linear_assignment/__init__.py 100.00% <0.00%> (ø)
python/cugraph/structure/number_map.py 55.95% <0.00%> (+0.11%) ⬆️

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 1be056b...e283aa2. Read the comment docs.

@BradReesWork BradReesWork added the improvement Improvement / enhancement to an existing function label Nov 13, 2020
@BradReesWork BradReesWork added this to the 0.17 milestone Nov 13, 2020
@aschaffer aschaffer requested a review from a team as a code owner November 19, 2020 23:59
@aschaffer aschaffer requested a review from a team as a code owner November 20, 2020 00:04
Comment thread cpp/CMakeLists.txt
Comment thread build.sh
@aschaffer
Copy link
Copy Markdown
Collaborator Author

Rerun tests.

Copy link
Copy Markdown
Collaborator

@ChuckHastings ChuckHastings left a comment

Choose a reason for hiding this comment

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

A few things I think we should consider excluding. Otherwise I can't wait to have this merged.

Comment thread cpp/include/experimental/eidecl_graph.hpp
Comment thread cpp/include/eidecl_graph.hpp
Comment thread cpp/include/eidir_graph.hpp Outdated
Comment thread cpp/include/graph.hpp
Comment thread cpp/src/community/louvain.cu
@aschaffer aschaffer changed the title [WIP] Enhance compile time [REVIEW] Enhance compile time Nov 23, 2020
@BradReesWork BradReesWork linked an issue Nov 23, 2020 that may be closed by this pull request
5 tasks
@BradReesWork BradReesWork merged commit 1952767 into rapidsai:branch-0.17 Nov 23, 2020
@aschaffer aschaffer deleted the enh_ext_compile_time_017 branch May 6, 2021 15:34
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ENH] Reduce compile time

7 participants