Closed gawng closed 3 years ago
I'm not 100% convinced we want to manually update the ADFirstAidKit stuff. A couple options here:
Let's talk about this during the next maintenance meeting maybe.
Merging #53 (e911a6b) into master (136b3bb) will decrease coverage by
0.06%
. The diff coverage isn/a
.:exclamation: Current head e911a6b differs from pull request most recent head f43c669. Consider uploading reports for the commit f43c669 to get more accurate results
@@ Coverage Diff @@
## master #53 +/- ##
==========================================
- Coverage 46.13% 46.06% -0.07%
==========================================
Files 6 6
Lines 776 775 -1
==========================================
- Hits 358 357 -1
Misses 418 418
Impacted Files | Coverage Δ | |
---|---|---|
idwarp/MExt.py | 97.36% <0.00%> (-0.07%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 136b3bb...f43c669. Read the comment docs.
How does this look @eirikurj @bernardopacini? I added a new flag for F77 code which omits the standard specification. The flags can probably be simplified (we probably don't need PETSc/CGNS since it's just the Tapenade code).
I also see that @gang525 updated the complexify
module. I assume this is unavoidable since we do not have a centralized complexify package (yet) and this is OK here. We will of course have to do the same thing for other repos that use the same module (but potentially have slightly different code).
Purpose
The Fortran standard is 2008 because petsc wasn't cooperating with earlier standards.
Type of change
What types of change is it? Select the appropriate type(s) that describe this PR
Testing
I do not have complex IDWarp compiled, so those tests were skipped.
Checklist
Put an
x
in the boxes that apply.flake8
andblack
to make sure the code adheres to PEP-8 and is consistently formatted