-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Reduce binary size for Linux and Mac package #3337
Conversation
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
.github/workflows/release_mac.yml
Outdated
@@ -4,7 +4,7 @@ on: | |||
push: | |||
branches: [master, rel-*] | |||
pull_request: | |||
branches: [rel-*] | |||
branches: [rel-*, master] # TODO: remove it before merge | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mac CI fails because of this pypa/setuptools-scm#542. Will try to reduce binary size as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ONNX decided to deprecate python 3.5 in the following 1.9. Will remove it in this PR to prevent failure caused by Python 3.5 incompatibility
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
.github/workflows/release_mac.yml
Outdated
@@ -39,6 +39,8 @@ jobs: | |||
- name: Install Python dependencies | |||
run: | | |||
python -m pip install -q --upgrade pip | |||
# Latest setuptools-scm-6.0.1 (dependency of setuptools) does not support Python 3.5 anymore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still relevant if we want to remove support for Python 3.5?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. Removed. Thanks
can we separate this in 2 PRs... to keep things clean.... Create a new 1 for python deprecation and keep this for binary size reduction. Thanks! |
Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
Sure thing. A new PR (#3353) decouple from this PR |
Description
Linux package -- 18.9MB ->11.5MB
Mac package -- 12MB -> 11.3MB
Meanwhile; the total running time will be much faster as well: (Linux 59mins -> 8 mins; Mac 26mins -> 9mins)
Motivation
ONNX 1.8.1 PyPI package becomes much larger than 1.8.0, especially Linux package. The main reason is ONNX upgraded its protobuf-compiler version from 2.6 to 3.11. We need to figure out a reasonable way to shrink protobuf dependency even though it needs to be upgraded.