Looks mostly okay, but I think we need to rethink the initial mass calculation before ...
15 years, 10 months ago
(2009-05-05 16:32:29 UTC)
#1
Looks mostly okay, but I think we need to rethink the initial mass calculation
before it gets committed.
http://codereview.appspot.com/59041/diff/1/2
File HaloFinding.py (right):
http://codereview.appspot.com/59041/diff/1/2#newcode197
Line 197:
Not sure we need this?
http://codereview.appspot.com/59041/diff/1/2#newcode433
Line 433: padded, LE, RE, self._data_source =
self._partition_hierarchy_3d(padding=self.padding)
We should get rid of this initial step of partitioning the entire hierarchy, and
instead move to using a DerivedQuantity. I'd say something like
all_data = self.pf.h.all_data()
all_data.quantities["TotalQuantity"]("ParticleMassMsun", lazy_reader=True)
which will automatically parallelize. This would get rid of most of the
following lines, and we could avoid the mpi_allsum, too. I'll need to add in
the preloading to the DQ object, however.
http://codereview.appspot.com/59041/diff/1/2#newcode450
Line 450: self._data_source.hierarchy.queue)
Should be fine.
Issue 59041: HDF5 HaloFinder preloading
Created 15 years, 10 months ago by stephenskory
Modified 1 year, 4 months ago
Reviewers: matthewturk
Base URL: http://svn.enzotools.org/yt/trunk/yt/lagos/
Comments: 3