zhafen / linefinder

A tool for finding and classifying the worldlines of Lagrangian parcels of mass, in the context of hydrodynamic simulations of galaxy formation.
https://zhafen.github.io/linefinder
MIT License
0 stars 1 forks source link

Parallel galaxy finding results don't match serial galaxy finding results #17

Closed zhafen closed 7 years ago

zhafen commented 7 years ago

Originally reported by Zachary Hafen (Bitbucket: zhafen, GitHub: zhafen)


Running parallelized galaxy finding (using galaxy_find.ParticleTrackGalaxyFinder with n_processors>1) produces results that don't agree with serial galaxy finding. In particular the test in tests/test_galaxy_find.py::TestParticleTrackGalaxyFinderParallel fails.

I checked the serial results, and added some additional checks to make sure that we get the expected results at multiple snapshots, but all passed, so the serial code is still working as expected, and matches what would happen if we used GalaxyFinder by hand.

This is difficult to debug, because it requires using parallel debugging tools. However, since serial galaxy finding still works, and is sufficiently fast, this is not critical.


zhafen commented 7 years ago

Original comment by Zachary Hafen (Bitbucket: zhafen, GitHub: zhafen)


This was a typo in storing the results. Resolved as of commit b24a353.