search

adamantine-sim / adamantine

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

Add address sanitizer in CI and switch cpu build to release #85

Closed Rombur closed 3 years ago

Rombur commented 3 years ago

Not ready to be merged yet but that's the only way to test the CI.

codecov-io commented 3 years ago

Codecov Report

Merging #85 (e687fe5) into master (6914ea2) will decrease coverage by 0.38%. The diff coverage is 97.34%.

@@            Coverage Diff             @@
##           master      #85      +/-   ##
==========================================
- Coverage   92.63%   92.24%   -0.39%     
==========================================
  Files          27       27              
  Lines        1493     1316     -177     
==========================================
- Hits         1383     1214     -169     
+ Misses        110      102       -8
Impacted Files Coverage Δ
source/ThermalPhysics.hh 11.11% <ø> (-38.89%) :arrow_down:
application/adamantine.hh 65.78% <85.71%> (-2.46%) :arrow_down:
tests/test_material_deposition.cc 98.79% <98.77%> (-0.15%) :arrow_down:
tests/main.cc 100.00% <100.00%> (ø)
tests/test_thermal_physics.hh 100.00% <100.00%> (ø)
source/ImplicitOperator.hh 60.00% <0.00%> (-40.00%) :arrow_down:
source/Operator.hh 0.00% <0.00%> (-25.00%) :arrow_down:
source/utils.hh 77.77% <0.00%> (-5.56%) :arrow_down:
source/ThermalOperator.hh 46.66% <0.00%> (-0.71%) :arrow_down:
... and 18 more
Rombur commented 3 years ago

retest this please

Rombur commented 3 years ago

Retest this please

Rombur commented 3 years ago

This is ready for review. The problem with the build system was because there is an @ in the name of the directory generated by jenkins. I now filter out the @ with a regex.