Issue #25: Communication Partition Grid Bug

#226 Merged
Repository
enzo-dev
Branch
week-of-code
Author
  1. James Larrue
Reviewers

Comments (3)

  1. Sam Skillman

    Thanks @james_larrue , this looks great now! I'm going to decline the old PR, wait for this one to pass the push suite (just in case), and then approve. If @ngoldbaum could try it on his run where he specified the layout and @bwoshea can confirm that this fixes the np=4 test, we'll pull it in!

  2. Brian OShea

    OK, I verified that this works fine for the test run I was seeing, for np=1,2,4,8. I think we're all set, @samskillman .