Skip to content

[FIX] Remove -g from cython compile commands#1783

Merged
ajschmidt8 merged 3 commits intorapidsai:branch-21.08from
trxcllnt:fea/cython-no-debug
Aug 25, 2021
Merged

[FIX] Remove -g from cython compile commands#1783
ajschmidt8 merged 3 commits intorapidsai:branch-21.08from
trxcllnt:fea/cython-no-debug

Conversation

@trxcllnt
Copy link
Copy Markdown
Collaborator

Removes -g from the compile commands generated by distutils to compile Cython files.

This will make our container images, conda packages, and python wheels smaller.

Before this change:
$ shopt -s globstar; du -Shc cugraph/python/cugraph/**/*.so
7.6M	cugraph/python/cugraph/centrality/betweenness_centrality_wrapper.cpython-38-x86_64-linux-gnu.so
7.7M	cugraph/python/cugraph/centrality/edge_betweenness_centrality_wrapper.cpython-38-x86_64-linux-gnu.so
7.9M	cugraph/python/cugraph/centrality/katz_centrality_wrapper.cpython-38-x86_64-linux-gnu.so
5.6M	cugraph/python/cugraph/comms/comms_wrapper.cpython-38-x86_64-linux-gnu.so
7.7M	cugraph/python/cugraph/community/ecg_wrapper.cpython-38-x86_64-linux-gnu.so
8.7M	cugraph/python/cugraph/community/egonet_wrapper.cpython-38-x86_64-linux-gnu.so
7.2M	cugraph/python/cugraph/community/ktruss_subgraph_wrapper.cpython-38-x86_64-linux-gnu.so
7.8M	cugraph/python/cugraph/community/leiden_wrapper.cpython-38-x86_64-linux-gnu.so
7.9M	cugraph/python/cugraph/community/louvain_wrapper.cpython-38-x86_64-linux-gnu.so
8.7M	cugraph/python/cugraph/community/spectral_clustering_wrapper.cpython-38-x86_64-linux-gnu.so
7.2M	cugraph/python/cugraph/community/subgraph_extraction_wrapper.cpython-38-x86_64-linux-gnu.so
6.8M	cugraph/python/cugraph/community/triangle_count_wrapper.cpython-38-x86_64-linux-gnu.so
8.0M	cugraph/python/cugraph/components/connectivity_wrapper.cpython-38-x86_64-linux-gnu.so
6.9M	cugraph/python/cugraph/cores/core_number_wrapper.cpython-38-x86_64-linux-gnu.so
7.2M	cugraph/python/cugraph/cores/k_core_wrapper.cpython-38-x86_64-linux-gnu.so
7.7M	cugraph/python/cugraph/dask/centrality/mg_katz_centrality_wrapper.cpython-38-x86_64-linux-gnu.so
7.6M	cugraph/python/cugraph/dask/community/louvain_wrapper.cpython-38-x86_64-linux-gnu.so
7.5M	cugraph/python/cugraph/dask/components/mg_connectivity_wrapper.cpython-38-x86_64-linux-gnu.so
7.8M	cugraph/python/cugraph/dask/link_analysis/mg_pagerank_wrapper.cpython-38-x86_64-linux-gnu.so
7.5M	cugraph/python/cugraph/dask/structure/replication.cpython-38-x86_64-linux-gnu.so
7.6M	cugraph/python/cugraph/dask/traversal/mg_bfs_wrapper.cpython-38-x86_64-linux-gnu.so
7.6M	cugraph/python/cugraph/dask/traversal/mg_sssp_wrapper.cpython-38-x86_64-linux-gnu.so
7.9M	cugraph/python/cugraph/generators/rmat_wrapper.cpython-38-x86_64-linux-gnu.so
440K	cugraph/python/cugraph/internals/internals.cpython-38-x86_64-linux-gnu.so
8.0M	cugraph/python/cugraph/layout/force_atlas2_wrapper.cpython-38-x86_64-linux-gnu.so
8.0M	cugraph/python/cugraph/linear_assignment/lap_wrapper.cpython-38-x86_64-linux-gnu.so
7.1M	cugraph/python/cugraph/link_analysis/hits_wrapper.cpython-38-x86_64-linux-gnu.so
8.2M	cugraph/python/cugraph/link_analysis/pagerank_wrapper.cpython-38-x86_64-linux-gnu.so
7.5M	cugraph/python/cugraph/link_prediction/jaccard_wrapper.cpython-38-x86_64-linux-gnu.so
7.4M	cugraph/python/cugraph/link_prediction/overlap_wrapper.cpython-38-x86_64-linux-gnu.so
8.6M	cugraph/python/cugraph/sampling/random_walks_wrapper.cpython-38-x86_64-linux-gnu.so
7.5M	cugraph/python/cugraph/structure/graph_primtypes.cpython-38-x86_64-linux-gnu.so
9.0M	cugraph/python/cugraph/structure/graph_primtypes_wrapper.cpython-38-x86_64-linux-gnu.so
16M	cugraph/python/cugraph/structure/renumber_wrapper.cpython-38-x86_64-linux-gnu.so
7.1M	cugraph/python/cugraph/structure/utils_wrapper.cpython-38-x86_64-linux-gnu.so
8.0M	cugraph/python/cugraph/traversal/bfs_wrapper.cpython-38-x86_64-linux-gnu.so
8.0M	cugraph/python/cugraph/traversal/sssp_wrapper.cpython-38-x86_64-linux-gnu.so
7.5M	cugraph/python/cugraph/traversal/traveling_salesperson_wrapper.cpython-38-x86_64-linux-gnu.so
7.9M	cugraph/python/cugraph/tree/minimum_spanning_tree_wrapper.cpython-38-x86_64-linux-gnu.so
7.4M	cugraph/python/cugraph/utilities/path_retrieval_wrapper.cpython-38-x86_64-linux-gnu.so
120K	cugraph/python/cugraph/utilities/pointer_utils.cpython-38-x86_64-linux-gnu.so
306M	total
After this change:
$ shopt -s globstar; du -Shc cugraph/python/cugraph/**/*.so
660K	cugraph/python/cugraph/centrality/betweenness_centrality_wrapper.cpython-38-x86_64-linux-gnu.so
668K	cugraph/python/cugraph/centrality/edge_betweenness_centrality_wrapper.cpython-38-x86_64-linux-gnu.so
676K	cugraph/python/cugraph/centrality/katz_centrality_wrapper.cpython-38-x86_64-linux-gnu.so
408K	cugraph/python/cugraph/comms/comms_wrapper.cpython-38-x86_64-linux-gnu.so
660K	cugraph/python/cugraph/community/ecg_wrapper.cpython-38-x86_64-linux-gnu.so
696K	cugraph/python/cugraph/community/egonet_wrapper.cpython-38-x86_64-linux-gnu.so
608K	cugraph/python/cugraph/community/ktruss_subgraph_wrapper.cpython-38-x86_64-linux-gnu.so
668K	cugraph/python/cugraph/community/leiden_wrapper.cpython-38-x86_64-linux-gnu.so
676K	cugraph/python/cugraph/community/louvain_wrapper.cpython-38-x86_64-linux-gnu.so
724K	cugraph/python/cugraph/community/spectral_clustering_wrapper.cpython-38-x86_64-linux-gnu.so
600K	cugraph/python/cugraph/community/subgraph_extraction_wrapper.cpython-38-x86_64-linux-gnu.so
576K	cugraph/python/cugraph/community/triangle_count_wrapper.cpython-38-x86_64-linux-gnu.so
684K	cugraph/python/cugraph/components/connectivity_wrapper.cpython-38-x86_64-linux-gnu.so
584K	cugraph/python/cugraph/cores/core_number_wrapper.cpython-38-x86_64-linux-gnu.so
616K	cugraph/python/cugraph/cores/k_core_wrapper.cpython-38-x86_64-linux-gnu.so
636K	cugraph/python/cugraph/dask/centrality/mg_katz_centrality_wrapper.cpython-38-x86_64-linux-gnu.so
628K	cugraph/python/cugraph/dask/community/louvain_wrapper.cpython-38-x86_64-linux-gnu.so
620K	cugraph/python/cugraph/dask/components/mg_connectivity_wrapper.cpython-38-x86_64-linux-gnu.so
640K	cugraph/python/cugraph/dask/link_analysis/mg_pagerank_wrapper.cpython-38-x86_64-linux-gnu.so
652K	cugraph/python/cugraph/dask/structure/replication.cpython-38-x86_64-linux-gnu.so
628K	cugraph/python/cugraph/dask/traversal/mg_bfs_wrapper.cpython-38-x86_64-linux-gnu.so
632K	cugraph/python/cugraph/dask/traversal/mg_sssp_wrapper.cpython-38-x86_64-linux-gnu.so
684K	cugraph/python/cugraph/generators/rmat_wrapper.cpython-38-x86_64-linux-gnu.so
76K	cugraph/python/cugraph/internals/internals.cpython-38-x86_64-linux-gnu.so
684K	cugraph/python/cugraph/layout/force_atlas2_wrapper.cpython-38-x86_64-linux-gnu.so
680K	cugraph/python/cugraph/linear_assignment/lap_wrapper.cpython-38-x86_64-linux-gnu.so
592K	cugraph/python/cugraph/link_analysis/hits_wrapper.cpython-38-x86_64-linux-gnu.so
684K	cugraph/python/cugraph/link_analysis/pagerank_wrapper.cpython-38-x86_64-linux-gnu.so
632K	cugraph/python/cugraph/link_prediction/jaccard_wrapper.cpython-38-x86_64-linux-gnu.so
620K	cugraph/python/cugraph/link_prediction/overlap_wrapper.cpython-38-x86_64-linux-gnu.so
708K	cugraph/python/cugraph/sampling/random_walks_wrapper.cpython-38-x86_64-linux-gnu.so
624K	cugraph/python/cugraph/structure/graph_primtypes.cpython-38-x86_64-linux-gnu.so
752K	cugraph/python/cugraph/structure/graph_primtypes_wrapper.cpython-38-x86_64-linux-gnu.so
1.1M	cugraph/python/cugraph/structure/renumber_wrapper.cpython-38-x86_64-linux-gnu.so
612K	cugraph/python/cugraph/structure/utils_wrapper.cpython-38-x86_64-linux-gnu.so
672K	cugraph/python/cugraph/traversal/bfs_wrapper.cpython-38-x86_64-linux-gnu.so
680K	cugraph/python/cugraph/traversal/sssp_wrapper.cpython-38-x86_64-linux-gnu.so
644K	cugraph/python/cugraph/traversal/traveling_salesperson_wrapper.cpython-38-x86_64-linux-gnu.so
684K	cugraph/python/cugraph/tree/minimum_spanning_tree_wrapper.cpython-38-x86_64-linux-gnu.so
640K	cugraph/python/cugraph/utilities/path_retrieval_wrapper.cpython-38-x86_64-linux-gnu.so
24K	cugraph/python/cugraph/utilities/pointer_utils.cpython-38-x86_64-linux-gnu.so
26M	total

@trxcllnt trxcllnt requested a review from a team as a code owner August 19, 2021 19:46
@ajschmidt8 ajschmidt8 added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Aug 20, 2021
@trxcllnt trxcllnt added 2 - In Progress DO NOT MERGE Hold off on merging; see PR for details labels Aug 20, 2021
@trxcllnt trxcllnt added 3 - Ready for Review and removed 2 - In Progress DO NOT MERGE Hold off on merging; see PR for details labels Aug 20, 2021
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 21, 2021

Codecov Report

Merging #1783 (dddbd52) into branch-21.08 (bf64c2c) will decrease coverage by 0.11%.
The diff coverage is n/a.

❗ Current head dddbd52 differs from pull request most recent head 5dce5c2. Consider uploading reports for the commit 5dce5c2 to get more accurate results
Impacted file tree graph

@@               Coverage Diff                @@
##           branch-21.08    #1783      +/-   ##
================================================
- Coverage         59.85%   59.73%   -0.12%     
================================================
  Files                77       77              
  Lines              3547     3549       +2     
================================================
- Hits               2123     2120       -3     
- Misses             1424     1429       +5     
Impacted Files Coverage Δ
python/cugraph/community/__init__.py 70.37% <0.00%> (-18.52%) ⬇️
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 bf64c2c...5dce5c2. Read the comment docs.

@ajschmidt8 ajschmidt8 merged commit 8abf4ac into rapidsai:branch-21.08 Aug 25, 2021
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.

3 participants