search

flux-framework / flux-pmix

flux shell plugin to bootstrap openmpi v5+
GNU Lesser General Public License v3.0
2 stars 4 forks source link

plugin: use flux taskmap class to generate lpeers, procmap #66

Closed garlick closed 1 year ago

garlick commented 1 year ago

Problem: code for generating "maps" is no longer needed now that flux-core has taskmaps.

Use the new interface and throw away some code.

codecov[bot] commented 1 year ago

Codecov Report

Merging #66 (e1a5bb9) into main (e0d7cac) will decrease coverage by 0.24%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #66      +/-   ##
==========================================
- Coverage   78.46%   78.22%   -0.25%     
==========================================
  Files          10       10              
  Lines        1189     1162      -27     
==========================================
- Hits          933      909      -24     
+ Misses        256      253       -3
Impacted Files Coverage Δ
src/shell/plugins/maps.c 86.04% <ø> (-0.44%) :arrow_down:
src/shell/plugins/main.c 78.75% <100.00%> (-0.10%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more