<div dir="ltr">Hey Matt,<div><br></div><div>OK, so we *can* do it, but the question is, does it make sense to do it?  I thought it would be more consistent, but I'm not married to the idea.  Do you think it's worth the effort of modifying the code?  I'm just trying to look ahead to the future to when we have tons of different particle types, but maybe this is no big deal, in which case we can leave it as is.</div>

<div><br></div><div>Cameron</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 3, 2014 at 7:32 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 Cameron,<br>
<br>
I'd be fine with this change, but it touches a *lot* of user-facing<br>
code already.  If we did this, then inside _determine_fields and all<br>
the other places we guess the field names, we'd need backward<br>
compatibility.<br>
<br>
-Matt<br>
<div><div class="h5"><br>
On Thu, Jul 3, 2014 at 6:21 PM, Cameron Hummels <<a href="mailto:chummels@gmail.com">chummels@gmail.com</a>> wrote:<br>
> Hello everyone,<br>
><br>
> As I'm updating some of the documentation in 3.0 to be current on field<br>
> generation, I thought of something that might make the particle fields a bit<br>
> more clean as we build on them in the future.<br>
><br>
> As I understand it, each particle type from a given code will generate its<br>
> own namespace for those native particle quantities.  So for example a Gadget<br>
> binary dataset will read in its star, gas, and DM particles as:<br>
><br>
> ('star', 'particle_mass')<br>
> ('gas', 'particle_mass')<br>
> ('DM', 'particle_mass')<br>
><br>
> but when these different particles are deposited and smoothed on to the<br>
> grid, they all get put into the same 'deposit' namespace:<br>
><br>
> ('deposit', 'star_density')<br>
> ('deposit', 'gas_density')<br>
> ('deposit', 'DM_density')<br>
><br>
> It seems to me that perhaps we should create a separate deposit namespace<br>
> for each of the native particle types, so that we'll have a clean 1-to-1<br>
> conversion between native particle types and smoothed particle types in<br>
> namespaces.  Now the above fields would map to:<br>
><br>
> ('deposit_star', 'density')<br>
> ('deposit_gas', 'density')<br>
> ('deposit_DM', 'density')<br>
><br>
> This doesn't seem like it would be hard to change in the field-generation<br>
> infrastructure, but it might break things later on, which I've just not yet<br>
> considered.  Anyway, I just wanted to bounce this off of people.  It may not<br>
> be better in the long run, but it seemed like if we're breaking API, it<br>
> should be done now instead of later on.  I may be missing something big<br>
> here, but I wanted to see what others thought.<br>
><br>
> Cameron<br>
><br>
> --<br>
> Cameron Hummels<br>
> Postdoctoral Researcher<br>
> Steward Observatory<br>
> University of Arizona<br>
> <a href="http://chummels.org" target="_blank">http://chummels.org</a><br>
><br>
</div></div>> _______________________________________________<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>
_______________________________________________<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>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Cameron Hummels<div>Postdoctoral Researcher</div><div>Steward Observatory</div><div>University of Arizona</div><div><a href="http://chummels.org" target="_blank">http://chummels.org</a></div>


</div>