-
Notifications
You must be signed in to change notification settings - Fork 62
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
[ML] Upgrade PyTorch to version 1.13.1 #2430
Conversation
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.
LGTM
Thanks for doing this I know it is very time consuming
@@ -85,7 +87,8 @@ bool CCmdLineParser::parse(int argc, | |||
return false; | |||
} | |||
if (vm.count("version") > 0) { | |||
std::cerr << ver::CBuildInfo::fullInfo() << std::endl; | |||
std::cerr << "PyTorch Version " << TORCH_VERSION << std::endl |
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.
👍
@@ -249,14 +249,53 @@ sudo ./cmake-3.23.2-Linux-x86_64.sh --skip-license --prefix=/usr/local/cmake | |||
|
|||
Please ensure `/usr/local/cmake/bin` is in your `PATH` environment variable. | |||
|
|||
### OpenSSL | |||
|
|||
Python 3.10 requires OpenSSL 1.1. No other version is acceptable. |
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.
No OpenSSL 3.0?
OpenSSL 1.1.1 is LTS until 11th September 2023, hopefully we can update next time
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.
Yes, next time we upgrade Python presumably a different version will be needed.
But it's important to remember that we only use OpenSSL to build Python and we only use Python to build PyTorch. It's not like we use them in every build or ship them. As long as the version we've used is secure on the day we build PyTorch it's good enough.
Upgrade to the latest version of PyTorch 1, which is 1.13.1.
The version of Python used for the build is also upgraded, from 3.7 to 3.10. Python 3.10 runs natively on macOS on ARM, and this makes the macOS ARM build much less fiddly and error-prone.