[Yt-dev] Particle IO Disabled for now
Matthew Turk
matthewturk at gmail.com
Fri Dec 17 14:58:00 PST 2010
Hi all,
I have disabled fast ParticleIO for now. There's a bug in this code,
which I am struggling to figure out the right way to fix. This only
applies to Enzo data, and the bug only showed up when you had a
derived field depending on only one field, and only if it's more than
a simple conversion factor. The only field I found that this applied
to was ParticleAge.
During the reading, we pass in conversion factors for fields, so that
when returned they will be correctly scaled. However, I was
incorrectly setting dependency fields as the main field; this means
that, for instance, the ParticleAge conversion was being applied to
the creation_time field, and then creation_time was set as
ParticleAge. This is inside yt/data_objects/particle_io.py.
The problem here is getting all three things playing nicely:
* Particle IO
* Conversions
* Derived fields based on particles
I have to sit back and think about ParticleIO. Really I have always
kind of hated my particle IO implementation, but I think the solution
is to simplify it and make it behave more straightforwardly. I'll
notify the list when I've fixed the bug and re-enabled ParticleIO.
Best,
Matt
More information about the yt-dev
mailing list