<p dir="ltr">Okay. Y'all convinced me. Let's revert and fix it to be consistent.</p>
<div class="gmail_quote">On Dec 3, 2013 7:28 AM, "Kacper Kowalik" <<a href="mailto:xarthisius.kk@gmail.com">xarthisius.kk@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 12/02/2013 09:53 PM, Nathan Goldbaum wrote:<br>
> Here is the change:<br>
><br>
> <a href="https://bitbucket.org/ngoldbaum/yt/commits/2e6d536ffa1976e4699093c4bb3b3bc4ade3f9da" target="_blank">https://bitbucket.org/ngoldbaum/yt/commits/2e6d536ffa1976e4699093c4bb3b3bc4ade3f9da</a><br>
><br>
> We'll definitely be leaving 'm' -> 'milli' alone.  FWIW, we already do this<br>
> by mapping 'mpc' -> 'Mpc' in plot labels.<br>
<br>
That's precisely the reason it shouldn't be done this way. You're<br>
writing additional lines of code to implement workaround for invalid<br>
exception from general rules.<br>
<br>
I'd give strong -1 to the idea and additionally s/mpc/Mpc/g through out<br>
the codebase. Of course bear in mind that this comes from a person that<br>
never did simulation larger than 20kpc :P<br>
Cheers,<br>
Kacper<br>
<br>
><br>
> On Mon, Dec 2, 2013 at 3:46 PM, Britton Smith <<a href="mailto:brittonsmith@gmail.com">brittonsmith@gmail.com</a>>wrote:<br>
><br>
>> Ok, then I will put my anal personality (which would never agree to such a<br>
>> thing) aside and say I'm cool with this.<br>
>><br>
>><br>
>> On Mon, Dec 2, 2013 at 8:43 PM, Matthew Turk <<a href="mailto:matthewturk@gmail.com">matthewturk@gmail.com</a>>wrote:<br>
>><br>
>>> Hi Britton,<br>
>>><br>
>>> This would be in the units refactor of yt-3.0, and would *only* apply<br>
>>> to parsecs.  This would be the exclusive special case of m[some unit]<br>
>>> meaning Mega.<br>
>>><br>
>>> -Matt<br>
>>><br>
>>> On Mon, Dec 2, 2013 at 3:41 PM, Britton Smith <<a href="mailto:brittonsmith@gmail.com">brittonsmith@gmail.com</a>><br>
>>> wrote:<br>
>>>> Hi Matt,<br>
>>>><br>
>>>> Where exactly is this?  Is this in yt-3.0, and if so, would this apply<br>
>>> to<br>
>>>> all uses of m as milli?  Is that is so, then my concern is for when yt<br>
>>>> becomes more used by other domains that actually do have a valid use for<br>
>>>> milli.<br>
>>>><br>
>>>> Britton<br>
>>>><br>
>>>><br>
>>>> On Mon, Dec 2, 2013 at 8:24 PM, Matthew Turk <<a href="mailto:matthewturk@gmail.com">matthewturk@gmail.com</a>><br>
>>> wrote:<br>
>>>>><br>
>>>>> Hi all,<br>
>>>>><br>
>>>>> Nathan and I are working on some unit things.  One that came up is<br>
>>>>> that it looked like the conversion to mpc was wrong.<br>
>>>>><br>
>>>>> Turns out, it's not.  But what it's set up as is "mpc" for<br>
>>>>> "milliparsecs" and "Mpc" for "Megaparsecs."  We'd like to get rid of<br>
>>>>> any usage of milliparsecs, since I think the number of people who want<br>
>>>>> to use that (globally) are probably between 0 and 0, and just make Mpc<br>
>>>>> and mpc the same thing.<br>
>>>>><br>
>>>>> If anybody objects, write back.  Silence will be taken as tacit<br>
>>> agreement!<br>
>>>>><br>
>>>>> -Matt<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>
>>>><br>
>>>><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>
>>>><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>
>>><br>
>><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>
>><br>
>><br>
><br>
><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>
><br>
<br>
<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>
<br></blockquote></div>