Closed A-CGray closed 1 year ago
Merging #22 (357ac91) into main (64b7522) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## main #22 +/- ##
=======================================
Coverage 78.50% 78.50%
=======================================
Files 4 4
Lines 563 563
=======================================
Hits 442 442
Misses 121 121
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Purpose
I found that I got a ~2x speedup from replacing
-O2
with-Ofast -march=native
in my build config and the regression tests against reference values still passed. I think we probably don't want to make these the default flags becauseOfast
in unsafe andmarch=native
might mess with the conda build but I added a note to the docs for people who are building CMPLXFOIL locally themselves.Expected time until merged
1 day
Type of change
Testing
Checklist
flake8
andblack
to make sure the Python code adheres to PEP-8 and is consistently formattedfprettify
or C/C++ code withclang-format
as applicable