<div dir="ltr">Hey Doug,<div><br></div><div>I just want to echo Britton's comment about issuing a PR.  It's usually much easier to have these sorts of conversations if we're all looking at the same diff.</div><div>

<br></div><div>-Nathan</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 10, 2013 at 11:26 AM, Douglas Harvey Rudd <span dir="ltr"><<a href="mailto:drudd@uchicago.edu" target="_blank">drudd@uchicago.edu</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div style="word-wrap:break-word">
As a followup, I believe there is at least once instance where we would not want to use float64 as the accumulator: if the field itself is integer and the sum does not overflow.  Something like this might be better:
<div><br>
</div>
<div>if 'int' in str(data[field].dtype) :</div>
<div>   totals.append(data[field].sum(dtype=np.int64))</div>
<div>elif 'float' in str(data[field].dtype) :</div>
<div>   totals.append(data[field].sum(dtype=np.float64))</div>
<div>else :</div>
<div>  totals.append(data[field].sum())</div>
<div><br>
</div>
<div>Using np.float64 in the case of integers as well would alleviate the overflow problem, but would lead to approximate results where the user might expect an exact answer.</div>
<div><br>
</div>
<div><div class="im">
<div>
<div style="text-indent:0px;letter-spacing:normal;font-variant:normal;text-align:-webkit-auto;font-style:normal;font-weight:normal;line-height:normal;text-transform:none;font-size:medium;white-space:normal;font-family:Helvetica;word-wrap:break-word;word-spacing:0px">


<div>Douglas Rudd</div>
<div>Scientific Computing Consultant<br>
Research Computing Center<br>
</div>
<div><a href="mailto:drudd@uchicago.edu" target="_blank">drudd@uchicago.edu</a></div>
</div>
<br>
<br>
</div>
<br>
</div><div><div class="h5"><div>
<div>On Jul 10, 2013, at 6:26 AM, Britton Smith <<a href="mailto:brittonsmith@gmail.com" target="_blank">brittonsmith@gmail.com</a>> wrote:</div>
<br>
<blockquote type="cite">
<div dir="ltr">
<div>
<div>Hi Doug,<br>
<br>
</div>
I don't seem any reason why we wouldn't want to fix this to always use higher precision.  Could you issue a PR?<br>
<br>
</div>
Britton<br>
</div>
<div class="gmail_extra"><br>
<br>
<div class="gmail_quote">On Wed, Jul 10, 2013 at 2:36 AM, Douglas Harvey Rudd <span dir="ltr">
<<a href="mailto:drudd@uchicago.edu" target="_blank">drudd@uchicago.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi all,<br>
<br>
I'm working on getting the ARTIO frontend working with a script used by the AGORA collaboration to measure density profiles.  This script uses the "TotalQuantity" quantity to sum all particle masses within a given sphere, then differences a nested set of spheres
 to measure the mean density in spherical shells.<br>
<br>
I noticed a 2% difference at large radius from my own profiling routines, and tracked it down to the precision of the sum, which used the same precision as the input variable, float32 for ARTIO. Since there are a large number of particles, summing 1e5 x 10^-7
 floats led to O(1%) error.<br>
<br>
While this is clearly not an ideal way to measure density profiles, it's probably a good idea to help avoid such issues in future by changing TotalQuantity (and other places, like _TotalMass) to use a 64-bit accumulator, e.g.<br>


        totals.append(data[field].sum(dtype=np.float64))<br>
<br>
The alternative is to promote all float32 input variables to float64, which is wasteful of memory, or otherwise warn the user that they are likely to encounter loss of precision.<br>
<br>
Douglas Rudd<br>
Scientific Computing Consultant<br>
Research Computing Center<br>
<a href="mailto:drudd@uchicago.edu" target="_blank">drudd@uchicago.edu</a><br>
<br>
<br>
<br>
_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org" target="_blank">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
</blockquote>
</div>
<br>
</div>
_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org" target="_blank">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
</blockquote>
</div>
<br>
</div></div></div>
</div>

<br>_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
<br></blockquote></div><br></div>