Skip to content
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

CI: Windows + CUDA #1585

Open
wants to merge 23 commits into
base: development
Choose a base branch
from
Open

CI: Windows + CUDA #1585

wants to merge 23 commits into from

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Dec 1, 2020

Summary

This a CI entry for Windows MSVC + CUDA.

@robertmaynard @WeiqunZhang @mic84

Additional background

The install of CUDA takes about 10min, but we cache its install after the first run (this cache will be evicted after 7 days of inactivity).

Checklist

The proposed changes:

  • fix a bug or incorrect behavior in AMReX
  • add new capabilities to AMReX
  • changes answers in the test suite to more than roundoff level
  • are likely to significantly affect the results of downstream AMReX users
  • are described in the proposed changes to the AMReX documentation, if appropriate

.github/workflows/windows.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Outdated Show resolved Hide resolved
@ax3l ax3l force-pushed the ci-winCUDA branch 8 times, most recently from 10210ac to 8d538eb Compare December 8, 2020 19:24
.github/workflows/windows.yml Outdated Show resolved Hide resolved
@ax3l
Copy link
Member Author

ax3l commented Mar 19, 2021

@robertmaynard not sure if relevant to your tasks, but we enabled .dll support in #1847 .

With this CI build (Windows + CUDA), I am still puzzled.

@sayerhs
Copy link
Contributor

sayerhs commented Mar 19, 2021

@ax3l May be add a cron option to on: to compile the Windows+CUDA build on a weekly basis?

@robertmaynard
Copy link
Contributor

@robertmaynard not sure if relevant to your tasks, but we enabled .dll support in #1847 .

With this CI build (Windows + CUDA), I am still puzzled.

I think you want the Ninja generator correct? But the CI config doesn't have an explicit Generator specified so you are getting Visual Studio ( I think that is the problem ).

@ax3l
Copy link
Member Author

ax3l commented Mar 19, 2021

I tried cmd terminal and -G Ninja in an earlier commit (2nd last), but that did not work either.

@ax3l ax3l force-pushed the ci-winCUDA branch 2 times, most recently from 4c188b0 to c092b4b Compare March 19, 2021 23:12
@ax3l
Copy link
Member Author

ax3l commented Mar 19, 2021

May be add a cron option to on: to compile the Windows+CUDA build on a weekly basis?

I am caching the download, but if it gets to build at some point and if it still runs a long time, then we should definitely do that.

.github/workflows/windows.yml Outdated Show resolved Hide resolved
@ax3l ax3l changed the title [WIP] CI: Windows + CUDA CI: Windows + CUDA Mar 25, 2021
@ax3l ax3l requested a review from mic84 April 1, 2021 22:44
@ax3l ax3l force-pushed the ci-winCUDA branch 2 times, most recently from 24169e8 to e197b73 Compare April 11, 2024 00:40
@ax3l ax3l force-pushed the ci-winCUDA branch 6 times, most recently from a2305ab to d8511e9 Compare April 11, 2024 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants