-
Notifications
You must be signed in to change notification settings - Fork 633
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Implementation of chunk balancer feature (#1775)
* Implementation of chunk balancer feature CONTEXT: This feature uses empirical data about the timings of each MPI process to adjust the chunk sizes to achieve optimal load balancing for repeated simulations, such as iterations in an optimization run. Processes which are consistently slow and overworked will have their chunk sizes reduced for the next iteration of an optimization run, and underworked processes will handle larger chunks. This approach has the advantage that it implicitly incorporates variable load between different machines running the MPI processes. SCOPE: - Added abstract class for chunk balancer - Added DefaultChunkBalancer implementation, which adjusts chunk sizes according to per-process working time (ignoring all-all comms) while maintaining previous split directions - Wrote unit tests for DefaultChunkBalancer to check for improvement in load-balancing, convergence to a load-balanced state, and that split_pos values are adjusted correctly for each iteration. - Wrote unit tests for the MockSimulation class used by the DefaultChunkBalancerTests - Added a binary_partition_utils.py library which includes lots of tree traversal algorithms which are useful to have for the chunk balancer - Wrote unit tests for binary_partition_utils.py * fixed wrong test filename in Makefile.am * add new modules to Makefile.am to be included in __init__.py * bugfix in tests * more test bugfixes * avoid merge conflict * refactored class names, added documentation for adjusting chunk layouts between runs of a program * update to Parallel_Meep.md for missing chunk layout file * test bugfix
- Loading branch information
1 parent
cc626e2
commit d12e544
Showing
12 changed files
with
1,641 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,4 +14,5 @@ Alec Hammond <[email protected]> | |
Yidong Chong <[email protected]> | ||
Robin Dunn <[email protected]> | ||
Ian Williamson <[email protected]> | ||
Andreas Hoenselaar <[email protected]> | ||
Andreas Hoenselaar <[email protected]> | ||
Ben Bartlett <[email protected]> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.