-
Notifications
You must be signed in to change notification settings - Fork 2.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
Replace inline pip install with pip install from requirements*.txt #21106
base: main
Are you sure you want to change the base?
Conversation
wheel | ||
pytest | ||
numpy==1.24.0 ; python_version < '3.12' | ||
numpy==1.26.0 ; python_version >= '3.12' |
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.
I have a PR(#21085) that updates Numpy to a newer version. Could you wait it? I think we have conflicts here.
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.
update to numpy >= 1.19.0
, same as your PR.
tools/ci_build/build.py
Outdated
dep_packages.append("cerberus") | ||
dep_packages.append("psutil") | ||
run_subprocess([sys.executable, "-m", "pip", "install", *dep_packages]) | ||
def install_python_deps(requirements_file='requirements.txt',numpy_version=None): |
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.
Or maybe we can totally remove this function.
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.
Do we want to keep the NumPy version overwritten capability? We can remove the function unless we don't want this capability.
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.
I think we no longer need to.
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.
We could just use the latest numpy
@@ -260,9 +260,6 @@ def convert_arg_line_to_args(self, arg_line): | |||
"--wheel_name_suffix", | |||
help="Suffix to append to created wheel names. This value is currently only used for nightly builds.", | |||
) | |||
parser.add_argument( |
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.
The only place that uses this argument was from line 2813, which is no longer needed.
Description
Replace inline pip install with pip install from requirements*.txt
Motivation and Context
so that CG can recognize
Dependency