Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Conversation

@lebeg
Copy link
Contributor

@lebeg lebeg commented Sep 14, 2018

Description

Used jom for windows builds since MSBuild is not able to parallelize nvcc calls.

Redone #12533

@lebeg lebeg force-pushed the fix-windows-build-speed branch 2 times, most recently from a85180d to 34023fd Compare September 14, 2018 11:03
@kalyc
Copy link
Contributor

kalyc commented Sep 14, 2018

Thank you for contributing @lebeg
Noticed that the tests are failing

@mxnet-label-bot[pr-awaiting-testing]

@marcoabreu marcoabreu added the pr-awaiting-testing PR is reviewed and waiting CI build and test label Sep 14, 2018
@lebeg lebeg force-pushed the fix-windows-build-speed branch 3 times, most recently from 43b4a30 to 725c5fd Compare September 17, 2018 12:37
@lebeg lebeg force-pushed the fix-windows-build-speed branch 2 times, most recently from 0f2f24d to b998e17 Compare September 18, 2018 10:08
@lebeg lebeg force-pushed the fix-windows-build-speed branch from b998e17 to de8f074 Compare September 18, 2018 11:06
@marcoabreu
Copy link
Contributor

@anirudhacharya

This PR fixes that your coverage reports in R (and other languages) were not properly associated with your PR.

@lebeg
Copy link
Contributor Author

lebeg commented Sep 19, 2018

We got builds down to:

  • Build CPU windows - 6m 31s
  • Build GPU MKLDNN windows - 12m 11s
  • Build GPU windows - 10m 13s

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

pr-awaiting-testing PR is reviewed and waiting CI build and test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants