[yt-dev] RFC: Change in behavior for pf.current_time in Enzo frontend

Britton Smith brittonsmith at gmail.com
Thu Nov 1 12:12:03 PDT 2012


I am +1 for yt 3.0, but -1 on changing it for 2.x.  The behavior of
pf.current_time is in the core of a ton of my own analysis and my guess is
that is true for many people.  In my opinion, changes like this that are
for the sake of making things more proper, but break with previous
behavior, should only be made in big version jumps, where in some sense,
all bets are off anyway.
This change will also need to be propagated through to things like the
SimulationTimeSeries.

Britton

On Thu, Nov 1, 2012 at 2:54 PM, Matthew Turk <matthewturk at gmail.com> wrote:

> Hi all,
>
> Right now, amongst the confusing parameters available to an Enzo
> frontend parameter file, we have:
>
> pf["InitialTime"]
> pf.current_time
>
> These two are right now exactly the same.  However, in most/all of the
> other frontends, pf.current_time is (or should be) the actual time in
> seconds.  For Enzo this would mean multiplying by pf["seconds"].
>
> Changing this could adversely affect many user scripts.  However, not
> changing it means we continue with this confusing behavior.  For
> instance, the TimeStamp callback is currently totally incorrect for
> Enzo, but completely correct for FLASH.
>
> Is this an important enough change to overcome the old behavior?  Or
> should this be deferred to 3.0?
>
> [+-][01]
>
> -Matt
> _______________________________________________
> 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/20121101/dd0a8711/attachment.html>


More information about the yt-dev mailing list