Skip to content

Commit

Permalink
#2201: addressed PR2203 NS reviww comments
Browse files Browse the repository at this point in the history
  • Loading branch information
ppebay authored and lifflander committed Nov 28, 2023
1 parent 899c8d7 commit 65feb2a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/vt/vrt/collection/balance/temperedlb/tempered_enums.h
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,8 @@ enum struct TransferTypeEnum : uint8_t {
/**
* \brief Form object clusters and attempt to perform swaps.
*
* Object can be clustered according to arbitrary definition, and swaps
* of entire clusters, according the nullset, between ranks are attempted.
* Object can be clustered including to arbitrary definition, and swaps
* of entire clusters, including the nullset, between ranks are attempted.
* This is especially useful when shared memory constraints are present,
* as breaking shared memory clusters results in higher overall memory
* footprint, in constrast with whole cluster swaps.
Expand Down
5 changes: 3 additions & 2 deletions src/vt/vrt/collection/balance/temperedlb/temperedlb.cc
Original file line number Diff line number Diff line change
Expand Up @@ -174,8 +174,9 @@ Default: AsyncInform
Values: {Original, Recursive, SwapClusters}
Default: Original
Description:
Transfer strategy to be used in transfer stage. Options are:
Original: transfer one object per transfer as in original Grapevine approach. Recursive: original strategy improved by recursion.
Transfer strategy to be used in transfer stage. Options are:
Original: transfer one object per transfer as in original Grapevine approach.
Recursive: original strategy improved by recursion.
When single object transfer is rejected, attempt to recurse in order to
pull more objects into the transfer and hereby minimize work added by
said transfer.
Expand Down

0 comments on commit 65feb2a

Please sign in to comment.