Skip to content

Enabled wheel packaging#3582

Merged
ilya-lavrenov merged 59 commits intoopenvinotoolkit:masterfrom
mryzhov:feature/pip_cmake
Feb 8, 2021
Merged

Enabled wheel packaging#3582
ilya-lavrenov merged 59 commits intoopenvinotoolkit:masterfrom
mryzhov:feature/pip_cmake

Conversation

@mryzhov
Copy link
Copy Markdown
Contributor

@mryzhov mryzhov commented Dec 11, 2020

Added cmake configuration to create wheel packages using cmake install.
All the wheel install logic moved to setup.py, cmake custom command just runs it during the build step

Steps for users:

  1. Configure project with enabled ENABLE_WHEEL option and all required libraries (IE, ngraph, plugins, python bindings)
  2. build ALL or particular target ie_wheel
  3. The wheel package will be available in the /bin directory

CI will set cmake variable to change package-specific metadata (license, description, version, etc..)

@mryzhov mryzhov added the category: Python API OpenVINO Python bindings label Dec 11, 2020
@mryzhov mryzhov requested a review from a team as a code owner December 11, 2020 12:54
@mryzhov mryzhov requested a review from a team December 11, 2020 12:54
@openvino-pushbot openvino-pushbot added the category: docs OpenVINO documentation label Dec 11, 2020
Comment thread inference-engine/ie_bridges/python/install/wheel/pypi_overview.md Outdated
Comment thread inference-engine/ie_bridges/python/install/wheel/setup.py.in Outdated
@dkurt
Copy link
Copy Markdown
Contributor

dkurt commented Dec 11, 2020

Here is our experiment with manylinux: https://github.com/dkurt/openvino-python-manylinux

Comment thread inference-engine/ie_bridges/python/CMakeLists.txt
Comment thread inference-engine/ie_bridges/python/install/wheel/CMakeLists.txt Outdated
Comment thread inference-engine/ie_bridges/python/install/wheel/CMakeLists.txt Outdated
Comment thread inference-engine/ie_bridges/python/install/wheel/CMakeLists.txt Outdated
@ilya-lavrenov ilya-lavrenov self-requested a review December 16, 2020 12:09
Comment thread inference-engine/ie_bridges/python/install/wheel/CMakeLists.txt Outdated
Comment thread inference-engine/ie_bridges/python/install/wheel/CMakeLists.txt Outdated
Comment thread inference-engine/ie_bridges/python/install/wheel/CMakeLists.txt Outdated
Comment thread inference-engine/ie_bridges/python/install/wheel/CMakeLists.txt Outdated
@asomsiko asomsiko self-requested a review December 21, 2020 09:14
Comment thread inference-engine/ie_bridges/python/install/wheel/CMakeLists.txt Outdated
Comment thread inference-engine/ie_bridges/python/CMakeLists.txt Outdated
@dkurt dkurt mentioned this pull request Jan 13, 2021
@mryzhov mryzhov requested review from a team January 27, 2021 09:35
@mryzhov mryzhov changed the title [WIP] Enabled wheel packaging Enabled wheel packaging Jan 27, 2021
@ilyachur ilyachur requested review from jiwaszki and removed request for a team February 1, 2021 03:55
Comment thread inference-engine/ie_bridges/python/wheel/pylint.log Outdated
@jiwaszki jiwaszki self-requested a review February 2, 2021 16:57
@mryzhov mryzhov added this to the 2021.3 milestone Feb 5, 2021
@mryzhov mryzhov requested review from IRDonch and removed request for a team February 5, 2021 11:48
@mryzhov mryzhov requested review from slyubimt and removed request for a team February 5, 2021 11:53
1) Setup.py can be run without cmake build
2) Wheel metadata can be set by environment variables
3) Cmake build uses configuration from .env file
@ilya-lavrenov ilya-lavrenov merged commit e2c67bd into openvinotoolkit:master Feb 8, 2021
@mryzhov mryzhov deleted the feature/pip_cmake branch February 26, 2021 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: docs OpenVINO documentation category: Python API OpenVINO Python bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants