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.

Sign up for GitHub

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

Jump to bottom

BLD Reduces size of wheels by stripping symbols #25123

Merged
merged 7 commits into from Dec 7, 2022
Merged

BLD Reduces size of wheels by stripping symbols #25123

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
4f62f3a
BLD Strip symbols
thomasjpfan Dec 6, 2022
173af77
CI [cd build]
thomasjpfan Dec 6, 2022
4aaa016
ENH Allow --debug to enable symbols [cd build gh]
thomasjpfan Dec 6, 2022
66b5507
DOC Minor rewording [cd build gh]
thomasjpfan Dec 6, 2022
0b9a92a
DOC Add more details [cd build gh]
thomasjpfan Dec 6, 2022
49b9465
ENH Adds SKLEARN_BUILD_ENABLE_DEBUG_SYMBOLS [cd build gh]
thomasjpfan Dec 7, 2022
ff2dc70
CLN Move closer to other flags [cd build gh]
thomasjpfan Dec 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
7 changes: 7 additions & 0 deletions doc/computing/parallelism.rst
View file
Open in desktop
Expand Up @@ -299,6 +299,13 @@ When this environment variable is set to a non zero value, the `Cython`
derivative, `boundscheck` is set to `True`. This is useful for finding
segfaults.

`SKLEARN_BUILD_ENABLE_DEBUG_SYMBOLS`
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

When this environment variable is set to a non zero value, the debug symbols
will be included in the compiled C extensions. Only debug symbols for POSIX
systems is configured.
Comment on lines +302 to +307
Copy link
Member

Choose a reason for hiding this comment

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

this doesn't really belong to parallelism.rst though.

Copy link
Member Author

Choose a reason for hiding this comment

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

Although this file is named parallelism.rst, the title is "Parallelism, resource management, and configuration". I'm guessing all the environment variables listed here fall under "configuration", which this PR adds onto.

Copy link
Member Author

Choose a reason for hiding this comment

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

As for discoverability, searching for "configuration" places the parallelism.rst page as second. I'm open to figuring out a better location for all the configuration options, but I do not think it is a blocker for this PR.


`SKLEARN_PAIRWISE_DIST_CHUNK_SIZE`
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Expand Down
13 changes: 13 additions & 0 deletions setup.py
View file
Open in desktop
Expand Up @@ -195,6 +195,19 @@ def build_extensions(self):
e.extra_compile_args += openmp_flag
e.extra_link_args += openmp_flag

build_with_debug_symbols = (
os.environ.get("SKLEARN_BUILD_ENABLE_DEBUG_SYMBOLS", "0") != "0"
)
if os.name == "posix":
if build_with_debug_symbols:
debug_compile_arg = ["-g"]
else:
# Setting -g0 will strip symbols, reducing the binary size of extensions
debug_compile_arg = ["-g0"]

for e in self.extensions:
e.extra_compile_args += debug_compile_arg

build_ext.build_extensions(self)

def run(self):
Expand Down