Hey Matt.<div><br></div><div>I installed yt-dev on Hopper and updated to Andrew's changes. The cellvolume test looks good, so the changes are good.</div><div><br></div><div>However, this brought up another issue with the Nyx reader. When I tried projecting in a 2.4 install and the fresh install, I got "OverflowError: value too large to convert to int" from calling read_and_seek. We should sort this out, but because it's in both versions, I don't think it has to do with these changes.</div>

<div><br></div><div>- Casey</div><div><br></div>
<div><br><div class="gmail_quote">On Thu, Sep 20, 2012 at 4:00 PM, Matthew Turk <span dir="ltr"><<a href="mailto:matthewturk@gmail.com" target="_blank">matthewturk@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Hi Casey,<br>
<br>
I think your and Andrew's intutions are correct.  Another good sanity<br>
check would be to check the before/after values of calculating:<br>
<br>
dd = pf.h.all_data()<br>
dd.quantities["TotalQuantity"]("CellVolume")<br>
<br>
I think as long as this is the sae, and the slices are all set, it<br>
should be all good.  The changes I think are quite good and should<br>
only change (by helping) at the level of just above roundoff error.<br>
<br>
-Matt<br>
<div><div><br>
On Thu, Sep 20, 2012 at 6:49 PM, Andrew Myers <<a href="mailto:atmyers@berkeley.edu" target="_blank">atmyers@berkeley.edu</a>> wrote:<br>
> Hi Casey,<br>
><br>
> Someone more knowledgeable than myself should confirm, but I would guess<br>
> that if I screwed up something with the 'dx' fields it would be fairly<br>
> blatant, such that a multi-level slice or projection would catch it. I've<br>
> been working with Orion data sets for about a week with this change, and it<br>
> seems fine, so I think we just need a basic sanity check for the other<br>
> frontends.<br>
><br>
> -Andrew Myers<br>
><br>
><br>
> On Thu, Sep 20, 2012 at 3:13 PM, Casey W. Stark <<a href="mailto:caseywstark@gmail.com" target="_blank">caseywstark@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Hey Matt.<br>
>><br>
>> The changes in the Nyx frontend look good to me, but you're right that<br>
>> I'll have to test it. Is it sufficient to compare multilevel slices before<br>
>> and after the changes?<br>
>><br>
>> - Casey<br>
>><br>
>><br>
>> On Thu, Sep 20, 2012 at 8:30 AM, Matthew Turk <<a href="mailto:matthewturk@gmail.com" target="_blank">matthewturk@gmail.com</a>><br>
>> wrote:<br>
>>><br>
>>> Hi all (but especially Casey and Tom Robitaille if he's still subscribed<br>
>>> :),<br>
>>><br>
>>> Andrew Myers found a very nice solution to issue #414:<br>
>>><br>
>>><br>
>>> <a href="https://bitbucket.org/yt_analysis/yt/issue/414/orion-ghot-zones-not-generating-correctly" target="_blank">https://bitbucket.org/yt_analysis/yt/issue/414/orion-ghot-zones-not-generating-correctly</a><br>



>>><br>
>>> which he's put in this PR:<br>
>>><br>
>>><br>
>>> <a href="https://bitbucket.org/yt_analysis/yt/pull-request/275/reading-in-dds-for-each-grid-from-the" target="_blank">https://bitbucket.org/yt_analysis/yt/pull-request/275/reading-in-dds-for-each-grid-from-the</a><br>



>>><br>
>>> I have only limited access to Nyx / Orion data with subgrids, so I<br>
>>> can't test this appropriately.  I think the changes look good,<br>
>>> however, so if I can get from any other boxlib user that they still<br>
>>> produce good results, I will accept them.  This is also a good time to<br>
>>> mention that going forward I hope to consolidate the various boxlib<br>
>>> readers.<br>
>>><br>
>>> Best,<br>
>>><br>
>>> Matt<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>
>><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>
>><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>
><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>
</div></div></blockquote></div><br></div>