Implements a distributed partitioner that is lazier than find_distributed_partitions - #277
Merged
Merged
Conversation
kaushikcfd
force-pushed
the
try_coarse_grained_dist_impl
branch
from
March 20, 2022 15:58
0eec71d to
adb0a65
Compare
kaushikcfd
marked this pull request as ready for review
March 20, 2022 16:34
kaushikcfd
force-pushed
the
try_coarse_grained_dist_impl
branch
2 times, most recently
from
March 20, 2022 18:51
1cf864c to
dcd1158
Compare
Collaborator
Author
|
With this branch: With main: That's for CNS 2D-DG operator, written at https://github.com/illinois-ceesd/drivers_y2-isolator. |
Owner
|
Whoa, nice job! It sure sounds like the lame annotations-based approach of #275 won't be needed now. |
3 tasks
kaushikcfd
force-pushed
the
try_coarse_grained_dist_impl
branch
from
March 20, 2022 23:05
dcd1158 to
4b66ba3
Compare
inducer
reviewed
Mar 21, 2022
kaushikcfd
force-pushed
the
try_coarse_grained_dist_impl
branch
from
March 24, 2022 05:55
4b66ba3 to
187fc9b
Compare
kaushikcfd
marked this pull request as draft
March 24, 2022 05:58
kaushikcfd
force-pushed
the
try_coarse_grained_dist_impl
branch
5 times, most recently
from
March 27, 2022 04:56
15f29f8 to
8a6e7bf
Compare
kaushikcfd
marked this pull request as ready for review
March 27, 2022 05:04
inducer
reviewed
Mar 29, 2022
Owner
|
Unsubscribing... @-mention or request review once it's ready for a look or needs attention. |
kaushikcfd
force-pushed
the
try_coarse_grained_dist_impl
branch
7 times, most recently
from
April 2, 2022 15:36
d72458d to
94a381d
Compare
inducer
force-pushed
the
try_coarse_grained_dist_impl
branch
from
April 3, 2022 01:18
94a381d to
0766823
Compare
inducer
reviewed
Apr 4, 2022
Co-authored-by: Andreas Kloeckner <inform@tiker.net>
kaushikcfd
force-pushed
the
try_coarse_grained_dist_impl
branch
2 times, most recently
from
April 4, 2022 20:26
07e21a0 to
ee304ae
Compare
The newer algorithm uses the following principle: - No node other than the ones indicated by the user must be materialized. - Be eager to schedule sends.
kaushikcfd
force-pushed
the
try_coarse_grained_dist_impl
branch
from
April 4, 2022 20:36
ee304ae to
596a8e3
Compare
Owner
|
This looks great! Thanks for working on it. |
This was referenced Nov 23, 2022
4 tasks
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
examples/distributed_partition_sends_asap.pywhich demonstrates the output for the partitioner.Draft because: