marcoabreu commented on a change in pull request #17962: Fix Windows GPU CI
URL: https://github.com/apache/incubator-mxnet/pull/17962#discussion_r403126540
##########
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:
Oh, they intentionally broke that backwards compatibility feature. Usually
it was possible to compile older cuda versions in later vs versions by
installing toolkits which make sure that the integration is available. Seems
like that caused issues and thus Microsoft and Nvidia decided to not go that
route any further.
In that case, find to proceed.
But is there still some kind of compatibility mode which checks that the is
still compliant with older cuda standards?
----------------------------------------------------------------
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
