Ticket #87 (closed bug: fixed)

Opened 4 years ago

Last modified 4 years ago

Load balancing changes answer

Reported by: rpwagner Owned by: jbordner
Priority: minor Milestone:
Component: enzo Version: devel svn
Keywords: Cc:

Description

-DENABLE_LOAD_BALANCE causes a change in the result. This was found by comparing the results of running an [example cosmology simulation]. [Changeset 2284] is the first placed where a vanilla build will show a difference in the result.

Unless someone has an argument for why load balancing should modify the results, this is a bug.

Change History

Changed 4 years ago by jbordner

  • owner changed from bwoshea to jbordner
  • component changed from documentation to enzo

I'll look into this for awhile, but may throw it back at Brian if I get bored. Grep shows the only difference is calling / not calling CommunicationLoadBalanceGrids?().

Changed 4 years ago by jbordner

The problem has been narrowed down to gravity. It works if gravity is turned off (grids are still load balanced and the results are bitwise identical), and the first change in any field occurs across the call to PrepareDensityField?().

Changed 4 years ago by jbordner

The first change seems to occur in the following function:

   EvolveLevel()
      PrepareDensityField()
         PrepareGravitatingMassField()
            CopyOverlappingParticleMassFields()
               grid::CheckForOverlap()
                  grid::AddOverlappingParticleMassField()

Changed 4 years ago by jbordner

  • status changed from new to closed
  • resolution set to fixed

David Collins found and fixed the bug! See SVN revision 2715 http://mngrid.ucsd.edu/projects/enzo/changeset/2715

Note: See TracTickets for help on using tickets.