Codecov Report

Merging #663 into development will increase coverage by 0.06%.
The diff coverage is 91.76%.

@@               Coverage Diff               @@
##           development     #663      +/-   ##
===============================================
+ Coverage         85.1%   85.17%   +0.06%     
===============================================
  Files              336      336              
  Lines            24927    24993      +66     
  Branches         11766    11824      +58     
===============================================
+ Hits             21215    21288      +73     
- Misses            3689     3690       +1     
+ Partials            23       15       -8

As you disable all of the traversial API, than why do we have this _children member at all? I think most users can live without this member. AFAIK, MPI does not have a child concept at all for its communicators and windows.

Good question. It doesn't seem to be used anywhere in the examples or applications and I don't see an immediate use-case to iterate over levels in the Team tree. My guess is that since the localities are hierarchical the teams were meant to be as well but I'm not sure that that is useful...

There is one reason though, the deconstructor, which also removes all children. I'm fine with this as is.