Skip to content

Commit

Permalink
#1359: LB: update documentation to reflect true threshold
Browse files Browse the repository at this point in the history
  • Loading branch information
lifflander committed Sep 28, 2021
1 parent 3af5300 commit b7d80fb
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 10 deletions.
9 changes: 4 additions & 5 deletions src/vt/vrt/collection/balance/greedylb/greedylb.cc
Original file line number Diff line number Diff line change
Expand Up @@ -81,11 +81,10 @@ GreedyLB::getInputKeysWithHelp() {
Values: <double>
Default: 0.8
Description:
The load threshold for objects to consider on each node. The default value of
0.8 will consider 80% of the average load for re-balancing on each node. The
order these will be selected is based on the value passed to "strategy". If
the parameter "auto" is set to "true", this will be the minimum threshold;
otherwise, it sets the threshold directly.
The load threshold of objects to consider for potential migration on each
rank. All objects over threshold * average_load on each rank will

This comment has been minimized.

Copy link
@nlslatt

nlslatt Sep 28, 2021

Collaborator

"will" --> "will be"

considered. If the parameter "auto" is set to "true", this will be the minimum
threshold; otherwise, it sets the threshold directly.
)"
},
{
Expand Down
10 changes: 5 additions & 5 deletions src/vt/vrt/collection/balance/hierarchicallb/hierlb.cc
Original file line number Diff line number Diff line change
Expand Up @@ -80,11 +80,11 @@ HierarchicalLB::getInputKeysWithHelp() {
Values: <double>
Default: 0.8
Description:
The load threshold for objects to consider on each node. The default value of
0.8 will consider 80% of the average load for re-balancing on each node. The
order these will be selected is based on the value passed to "strategy". If
the parameter "auto" is set to "true", this will be the minimum threshold;
otherwise, it sets the threshold directly.
The load threshold of objects to consider for potential migration on each
rank. All objects over threshold * average_load on each rank will

This comment has been minimized.

Copy link
@nlslatt

nlslatt Sep 28, 2021

Collaborator

"will" --> "will be"

considered. The order these will be selected is based on the value passed to
"strategy". If the parameter "auto" is set to "true", this will be the minimum
threshold; otherwise, it sets the threshold directly.
)"
},
{
Expand Down

0 comments on commit b7d80fb

Please sign in to comment.