leezu commented on a change in pull request #17962: Fix Windows GPU CI
URL: https://github.com/apache/incubator-mxnet/pull/17962#discussion_r403134909
##########
File path: ci/build_windows.py
##########
@@ -253,12 +269,8 @@ def main():
system = platform.system()
if system == 'Windows':
logging.info("Detected Windows platform")
- if 'OpenBLAS_HOME' not in os.environ:
- os.environ["OpenBLAS_HOME"] = "C:\\Program Files\\OpenBLAS-v0.2.19"
- if 'OpenCV_DIR' not in os.environ:
- os.environ["OpenCV_DIR"] = "C:\\Program
Files\\OpenCV-v3.4.1\\build"
if 'CUDA_PATH' not in os.environ:
- os.environ["CUDA_PATH"] = "C:\\Program Files\\NVIDIA GPU Computing
Toolkit\\CUDA\\v9.2"
+ os.environ["CUDA_PATH"] = "C:\\Program Files\\NVIDIA GPU Computing
Toolkit\\CUDA\\v10.2"
Review comment:
> But is there still some kind of compatibility mode which checks that the
is still compliant with older cuda standards?
No. Unix & CentOS tests Cuda 10.1. Windows now tests Cuda 10.2.
But the risk is low that we'd break Cuda 9 support within the next few days.
So it's not a one-way door decision. I suggest we discuss on dev if we want to
support cuda 9. If we decide to support it, let's switch the CentOS tests to
use Cuda 9.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
