Just as a note, I noticed a problem with the unit tests today and traced it back to a bug I introduced in the profiling code about two weeks ago. It's now fixed, but please be sure to update trunk to r1527.
---------- Forwarded message ---------- From: firstname.lastname@example.org Date: Mon, Nov 16, 2009 at 1:16 PM Subject: [Yt-svn] yt-commit r1527 - trunk/yt/lagos To: email@example.com
Author: mturk Date: Mon Nov 16 13:16:27 2009 New Revision: 1527 URL: http://yt.enzotools.org/changeset/1527
Log: This fixes a regression in the lazy profiling! For a very brief window of time, child cells were not being masked in the profiling method.
Modified: trunk/yt/lagos/Profiles.py ============================================================================== --- trunk/yt/lagos/Profiles.py (original) +++ trunk/yt/lagos/Profiles.py Mon Nov 16 13:16:27 2009 @@ -160,9 +160,14 @@ data =  for field in _field_mapping.get(this_field, (this_field,)): pointI = None - if check_cut and not self._data_source._is_fully_enclosed(source): + if check_cut: + # This conditional is so that we can have variable-length + # particle fields. Note that we can't apply the + # is_fully_enclosed to baryon fields, because child cells get + # in the way. if field in self.pf.field_info \ - and self.pf.field_info[field].particle_type: + and self.pf.field_info[field].particle_type \ + and not self._data_source._is_fully_enclosed(source): pointI = self._data_source._get_particle_indices(source) else: pointI = self._data_source._get_point_indices(source) _______________________________________________ Yt-svn mailing list Ytfirstname.lastname@example.org http://lists.spacepope.org/listinfo.cgi/yt-svn-spacepope.org