-
Notifications
You must be signed in to change notification settings - Fork 607
Add install_requirements.{sh,py} that only install requirements #7715
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
Merged
Merged
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
da1df94
Update
swolchok e411a1e
Update
swolchok 6e9745c
Update
swolchok 6d65549
Update
swolchok ac71f8d
Update
swolchok 00fd17d
Update
swolchok 9a1ddde
Update
swolchok 2063432
Update
swolchok cd79afd
Update
swolchok e76d22f
Update
swolchok a893391
Update
swolchok 84b8e5f
Update
swolchok fd0a5b1
Update
swolchok bf753a8
Update
swolchok 9158d0e
Update
swolchok 400ec37
Update
swolchok cab1ad3
Update
swolchok 1238f24
Update
swolchok e089ac5
Update
swolchok 2dcdbcd
Update
swolchok a89792a
Update
swolchok a95cc8d
Update
swolchok 27a3d64
Update
swolchok 1d4dfab
Update
swolchok ff3a43d
Update
swolchok f98d073
Update
swolchok 23c60d7
Update
swolchok 5e8be76
Update
swolchok 641ca89
Update
swolchok 57dab67
Update
swolchok 238d145
Update
swolchok 3f2c188
Update
swolchok 4561c6b
Update
swolchok c208b1f
Update
swolchok f2719b2
Update
swolchok 4b3f654
Update
swolchok a4f50f8
Update
swolchok 86e9c74
Update
swolchok 87a4c2d
Update
swolchok 5ca60ad
Update
swolchok b6da309
Update
swolchok File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,150 @@ | ||
# Copyright (c) Meta Platforms, Inc. and affiliates. | ||
# Copyright 2024-25 Arm Limited and/or its affiliates. | ||
# All rights reserved. | ||
# | ||
# This source code is licensed under the BSD-style license found in the | ||
# LICENSE file in the root directory of this source tree. | ||
|
||
import argparse | ||
import platform | ||
import re | ||
import subprocess | ||
import sys | ||
|
||
|
||
def python_is_compatible(): | ||
# Scrape the version range from pyproject.toml, which should be in the current directory. | ||
version_specifier = None | ||
with open("pyproject.toml", "r") as file: | ||
for line in file: | ||
if line.startswith("requires-python"): | ||
match = re.search(r'"([^"]*)"', line) | ||
if match: | ||
version_specifier = match.group(1) | ||
break | ||
|
||
if not version_specifier: | ||
print( | ||
"WARNING: Skipping python version check: version range not found", | ||
file=sys.stderr, | ||
) | ||
return False | ||
|
||
# Install the packaging module if necessary. | ||
try: | ||
import packaging | ||
except ImportError: | ||
subprocess.run( | ||
[sys.executable, "-m", "pip", "install", "packaging"], check=True | ||
) | ||
# Compare the current python version to the range in version_specifier. Exits | ||
# with status 1 if the version is not compatible, or with status 0 if the | ||
# version is compatible or the logic itself fails. | ||
try: | ||
import packaging.specifiers | ||
import packaging.version | ||
|
||
python_version = packaging.version.parse(platform.python_version()) | ||
version_range = packaging.specifiers.SpecifierSet(version_specifier) | ||
if python_version not in version_range: | ||
print( | ||
f'ERROR: ExecuTorch does not support python version {python_version}: must satisfy "{version_specifier}"', | ||
file=sys.stderr, | ||
) | ||
return False | ||
except Exception as e: | ||
print(f"WARNING: Skipping python version check: {e}", file=sys.stderr) | ||
return True | ||
|
||
|
||
# The pip repository that hosts nightly torch packages. | ||
TORCH_NIGHTLY_URL = "https://download.pytorch.org/whl/nightly/cpu" | ||
|
||
|
||
# Since ExecuTorch often uses main-branch features of pytorch, only the nightly | ||
# pip versions will have the required features. | ||
# | ||
# NOTE: If a newly-fetched version of the executorch repo changes the value of | ||
# NIGHTLY_VERSION, you should re-run this script to install the necessary | ||
# package versions. | ||
NIGHTLY_VERSION = "dev20250104" | ||
|
||
|
||
def install_requirements(use_pytorch_nightly): | ||
# pip packages needed by exir. | ||
EXIR_REQUIREMENTS = [ | ||
# Setting use_pytorch_nightly to false to test the pinned PyTorch commit. Note | ||
# that we don't need to set any version number there because they have already | ||
# been installed on CI before this step, so pip won't reinstall them | ||
f"torch==2.6.0.{NIGHTLY_VERSION}" if use_pytorch_nightly else "torch", | ||
( | ||
f"torchvision==0.22.0.{NIGHTLY_VERSION}" | ||
if use_pytorch_nightly | ||
else "torchvision" | ||
), # For testing. | ||
] | ||
|
||
EXAMPLES_REQUIREMENTS = [ | ||
f"torchaudio==2.6.0.{NIGHTLY_VERSION}" if use_pytorch_nightly else "torchaudio", | ||
] | ||
|
||
# Assemble the list of requirements to actually install. | ||
# TODO: Add options for reducing the number of requirements. | ||
REQUIREMENTS_TO_INSTALL = EXIR_REQUIREMENTS + EXAMPLES_REQUIREMENTS | ||
|
||
# Install the requirements. `--extra-index-url` tells pip to look for package | ||
# versions on the provided URL if they aren't available on the default URL. | ||
subprocess.run( | ||
[ | ||
sys.executable, | ||
"-m", | ||
"pip", | ||
"install", | ||
"-r", | ||
"requirements-examples.txt", | ||
*REQUIREMENTS_TO_INSTALL, | ||
"--extra-index-url", | ||
TORCH_NIGHTLY_URL, | ||
], | ||
check=True, | ||
) | ||
|
||
LOCAL_REQUIREMENTS = [ | ||
"third-party/ao", # We need the latest kernels for fast iteration, so not relying on pypi. | ||
] | ||
|
||
# Install packages directly from local copy instead of pypi. | ||
# This is usually not recommended. | ||
subprocess.run( | ||
[ | ||
sys.executable, | ||
"-m", | ||
"pip", | ||
"install", | ||
# Without --no-build-isolation, setup.py can't find the torch module. | ||
"--no-build-isolation", | ||
*LOCAL_REQUIREMENTS, | ||
], | ||
check=True, | ||
) | ||
|
||
|
||
def main(args): | ||
parser = argparse.ArgumentParser() | ||
parser.add_argument( | ||
"--use-pt-pinned-commit", | ||
action="store_true", | ||
help="build from the pinned PyTorch commit instead of nightly", | ||
) | ||
args = parser.parse_args(args) | ||
install_requirements(use_pytorch_nightly=not bool(args.use_pt_pinned_commit)) | ||
|
||
|
||
if __name__ == "__main__": | ||
import os | ||
|
||
# Before doing anything, cd to the directory containing this script. | ||
os.chdir(os.path.dirname(os.path.abspath(__file__))) | ||
if not python_is_compatible(): | ||
sys.exit(1) | ||
main(sys.argv[1:]) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
#!/bin/bash | ||
# Copyright (c) Meta Platforms, Inc. and affiliates. | ||
# All rights reserved. | ||
# | ||
# This source code is licensed under the BSD-style license found in the | ||
# LICENSE file in the root directory of this source tree. | ||
|
||
# Before doing anything, cd to the directory containing this script. | ||
cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null || /bin/true | ||
./run_python_script.sh ./install_requirements.py "$@" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
#!/bin/bash | ||
# Copyright (c) Meta Platforms, Inc. and affiliates. | ||
# All rights reserved. | ||
# | ||
# This source code is licensed under the BSD-style license found in the | ||
# LICENSE file in the root directory of this source tree. | ||
|
||
# Before doing anything, cd to the directory containing this script. | ||
cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null || /bin/true | ||
|
||
# Find the names of the python tools to use. | ||
if [[ -z $PYTHON_EXECUTABLE ]]; | ||
then | ||
if [[ -z $CONDA_DEFAULT_ENV ]] || [[ $CONDA_DEFAULT_ENV == "base" ]] || [[ ! -x "$(command -v python)" ]]; | ||
then | ||
PYTHON_EXECUTABLE=python3 | ||
else | ||
PYTHON_EXECUTABLE=python | ||
fi | ||
fi | ||
|
||
SCRIPT="$1"; shift | ||
$PYTHON_EXECUTABLE $SCRIPT "$@" | ||
|
||
# Exit with the same status as the python script. | ||
exit $? |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
np: don't you wanna do the other way around?
move the
install_requirements()
function into this file instead.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.
done