-
Notifications
You must be signed in to change notification settings - Fork 482
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Implement clamped Luus Jaakola search for function minimization. (#1690)
Summary: EmbeddingOffloadScaleupProposer attempts to find an approximately optimal trade-off between using extra cache memory to reduce prefetch delay and the additional plan cost resulting from a more challenging bin-packing of larger shards. Currently a binary search is used under the assumption that the cost would consistently decrease in the evaluated region. However, investigation revealed that the cost region can be non-smooth and multi-modal, leading to poor results with the binary search. This diff implements the Luus Jaakola search procedure which is much more robust and capable of navigating 'rough terrain'. In a subsequent diff EmbeddingOffloadScaleupProposer will be updated to utilize this approach instead of the binary search. See https://en.wikipedia.org/wiki/Luus-Jaakola for detail. Reviewed By: henrylhtsang Differential Revision: D52686075
- Loading branch information
1 parent
f934ec9
commit 7d5cce2
Showing
2 changed files
with
330 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
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