[yt-dev] Boxlib changes: RFT

[Casey W. Stark]

Hey Matt.

Really glad it's such a simple fix! I will try again with the update.

- Casey


On Wed, Sep 26, 2012 at 10:04 AM, Matthew Turk <matthewturk at gmail.com>wrote:

> Hi Casey,
>
> Ah, I saw this the other day when we were IO stress testing on EC2. It's a
> super simple fix of switching from int to int64 in the Cython IO code; I
> will push it today. For some reason I thought I already had! Thanks for the
> heads up.
>
> Matt
> On Sep 26, 2012 12:24 PM, "Casey W. Stark" <caseywstark at gmail.com> wrote:
>
>> Hey Matt.
>>
>> I installed yt-dev on Hopper and updated to Andrew's changes. The
>> cellvolume test looks good, so the changes are good.
>>
>> 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.
>>
>> - Casey
>>
>>
>> On Thu, Sep 20, 2012 at 4:00 PM, Matthew Turk <matthewturk at gmail.com>wrote:
>>
>>> Hi Casey,
>>>
>>> I think your and Andrew's intutions are correct.  Another good sanity
>>> check would be to check the before/after values of calculating:
>>>
>>> dd = pf.h.all_data()
>>> dd.quantities["TotalQuantity"]("CellVolume")
>>>
>>> I think as long as this is the sae, and the slices are all set, it
>>> should be all good.  The changes I think are quite good and should
>>> only change (by helping) at the level of just above roundoff error.
>>>
>>> -Matt
>>>
>>> On Thu, Sep 20, 2012 at 6:49 PM, Andrew Myers <atmyers at berkeley.edu>
>>> wrote:
>>> > Hi Casey,
>>> >
>>> > Someone more knowledgeable than myself should confirm, but I would
>>> guess
>>> > that if I screwed up something with the 'dx' fields it would be fairly
>>> > blatant, such that a multi-level slice or projection would catch it.
>>> I've
>>> > been working with Orion data sets for about a week with this change,
>>> and it
>>> > seems fine, so I think we just need a basic sanity check for the other
>>> > frontends.
>>> >
>>> > -Andrew Myers
>>> >
>>> >
>>> > On Thu, Sep 20, 2012 at 3:13 PM, Casey W. Stark <caseywstark at gmail.com
>>> >
>>> > wrote:
>>> >>
>>> >> Hey Matt.
>>> >>
>>> >> The changes in the Nyx frontend look good to me, but you're right that
>>> >> I'll have to test it. Is it sufficient to compare multilevel slices
>>> before
>>> >> and after the changes?
>>> >>
>>> >> - Casey
>>> >>
>>> >>
>>> >> On Thu, Sep 20, 2012 at 8:30 AM, Matthew Turk <matthewturk at gmail.com>
>>> >> wrote:
>>> >>>
>>> >>> Hi all (but especially Casey and Tom Robitaille if he's still
>>> subscribed
>>> >>> :),
>>> >>>
>>> >>> Andrew Myers found a very nice solution to issue #414:
>>> >>>
>>> >>>
>>> >>>
>>> https://bitbucket.org/yt_analysis/yt/issue/414/orion-ghot-zones-not-generating-correctly
>>> >>>
>>> >>> which he's put in this PR:
>>> >>>
>>> >>>
>>> >>>
>>> https://bitbucket.org/yt_analysis/yt/pull-request/275/reading-in-dds-for-each-grid-from-the
>>> >>>
>>> >>> I have only limited access to Nyx / Orion data with subgrids, so I
>>> >>> can't test this appropriately.  I think the changes look good,
>>> >>> however, so if I can get from any other boxlib user that they still
>>> >>> produce good results, I will accept them.  This is also a good time
>>> to
>>> >>> mention that going forward I hope to consolidate the various boxlib
>>> >>> readers.
>>> >>>
>>> >>> Best,
>>> >>>
>>> >>> Matt
>>> >>> _______________________________________________
>>> >>> yt-dev mailing list
>>> >>> yt-dev at lists.spacepope.org
>>> >>> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>>> >>
>>> >>
>>> >>
>>> >> _______________________________________________
>>> >> yt-dev mailing list
>>> >> yt-dev at lists.spacepope.org
>>> >> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>>> >>
>>> >
>>> >
>>> > _______________________________________________
>>> > yt-dev mailing list
>>> > yt-dev at lists.spacepope.org
>>> > http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>>> >
>>> _______________________________________________
>>> yt-dev mailing list
>>> yt-dev at lists.spacepope.org
>>> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>>>
>>
>>
>> _______________________________________________
>> yt-dev mailing list
>> yt-dev at lists.spacepope.org
>> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>>
>>
> _______________________________________________
> yt-dev mailing list
> yt-dev at lists.spacepope.org
> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.spacepope.org/pipermail/yt-dev-spacepope.org/attachments/20120926/2665e6ea/attachment.html>