Closed anilyil closed 11 months ago
Merging #81 (a7f36a5) into main (2c4769b) will increase coverage by
0.10%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #81 +/- ##
==========================================
+ Coverage 73.31% 73.42% +0.10%
==========================================
Files 6 6
Lines 757 760 +3
==========================================
+ Hits 555 558 +3
Misses 202 202
Files | Coverage Δ | |
---|---|---|
idwarp/MultiUnstructuredMesh.py | 84.84% <100.00%> (+0.17%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
I ran the multi us mesh with the OWN. It initializes and runs fine.
Purpose
The initialization of MultiUSMesh makes a few os.system calls to overwrite BCs. In my working docker container, this caused MPI problems where somehow the code was calling MPI init again from the os.system call. To avoid this, I changed the initialization to just load the cgns grid, overwrite BCs, and write the grid back out, which is what cgns_utils does anyways so this is more efficient too.
Expected time until merged
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