-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow for sparse OfflineLB maps that may be used with and LB configuration #2074
Comments
@lifflander I think the existing state of the code requires data for both the phase before (matching the pre-LB object distribution) and the phase after load balancing happens (for where the objects should be migrated). Are you suggesting a refactor that will allow migrating the objects to match the post-LB distribution without data specifying the starting distribution? I want to make sure the LB data file I give Arek with my LB config file will have the correct content. Would the specification you mentioned appear in the LB config file? What about something like |
After some discussion with @nlslatt, we are going to take this a little different direction to make the LB data file more clearly specified so it is not ambiguous. I'm opening a new issue to implement first: #2130 which will allow users to specify when phases are skipped or identical to the previous phase. |
…fflineLB is not configured properly
What Needs to be Done?
Currently, the LB data input files must be dense and fully specified for all phases.
LBDataRestartReader
to allow for sparse maps. (We need a proper specification for a phase that is the identical and thus left out.)vt_lb_name
is not set toOfflineLB
, theLBDataRestartReader
is not created. We should check the LB configuration file to see if any line has OfflineLB (in addition). If it is specified anywhere we should construct theLBDataRestartReader
. As another possible optimization, we only need the phases that are specified in the file to be read by the reader and those distributions stored.The text was updated successfully, but these errors were encountered: