adamantine-sim / adamantine

Software to simulate heat transfer for additive manufacturing
https://adamantine-sim.github.io/adamantine/
Other
39 stars 11 forks source link

Update CI #146

Closed Rombur closed 2 years ago

Rombur commented 2 years ago

This PR updates the Docker images and the CI. There are now two images: a CUDA image and a CPU only image. The new images contains Trilinos, ADOL-C, SymEngine, and dealii-weak_forms. Right now, dealii-weak_forms requires all the backends to be enabled (Sacado, ADOL-C, and SymeEngine). I'll remove them once dealii-weak_forms fixes that problem. I have added a CPU only build in the CI to make sure that everything works when we don't compile deal.II with CUDA support. I don't expect any difference but sometimes the compiler behaves slightly differently.

Rombur commented 2 years ago

There was an update in ArborX that breaks the code. I will fix it.

Rombur commented 2 years ago

This is ready for review. I have modified one test to make it more robust. We have one test where we do ray tracing and the ray was hitting the edge of the domain. When compiling with --fast-math the cell was missed, so I moved the ray to hit the cell further away from the edge. I have added a test using -O3 and --fast-math because we will need all the optimization we can get in production run. In production run, we should also add -march=native but this flag can be a problem for the CI because we don't know where it will run.