<div dir="ltr">That's true.  I'll make an issue for this so we deal with it after 3.0 is out.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 28, 2014 at 6:52 AM, Britton Smith <span dir="ltr"><<a href="mailto:brittonsmith@gmail.com" target="_blank">brittonsmith@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">For what it's worth, I agree that this merits another look in the future.  In principle, I like the two-argument syntax for create a region much better.  Python doesn't support overloading to my knowledge, but you can make a function accept different numbers of arguments like the following:<div>


<br></div><div><font face="courier new, monospace">def some_func(*args):</font></div><div><font face="courier new, monospace">    if len(args) == 2:</font></div><div><font face="courier new, monospace">        do something</font></div>


<div><font face="courier new, monospace">    elif len(args) == 3:</font></div><div><font face="courier new, monospace">        do something else</font></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra">

<br><br><div class="gmail_quote">
On Sat, Jul 26, 2014 at 3:20 AM, 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">


<p dir="ltr">-1. Too late in the cycle, but we can revisit (with compat layer) in the future.</p>
<div class="gmail_quote"><div><div>On Jul 25, 2014 8:40 PM, "Cameron Hummels" <<a href="mailto:chummels@gmail.com" target="_blank">chummels@gmail.com</a>> wrote:<br type="attribution"></div></div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
<div dir="ltr">As I'm writing up the objects documentation, I realize that we have some weirdness with our "region" object in that we require it to have a specified center, even though the only things which are really needed to specify a 3D box are the left corner and right corner.  I know in the past I've been perplexed by this additional center point requirement, and it might confuse others.  Since 3.0 is the time to break API, how do people feel about changing the constructor for the region object to just be:<div>





<br></div><div>region(left corner, right corner)</div><div><br></div><div>instead of the current constructor:</div><div><br></div><div>region(center, left corner, right corner)</div><div><br></div><div>?</div><div><br></div>





<div>I realize that this is right before the release, but it seems like if we're going to change it, now is the time to do it.  In fact, I was looking at ways to overload the region class with a wrapper so that we could just automatically accept 2 arguments and calculate the average of the left and right corners to pass to the 3-argument constructor.  But I couldn't seem to find a way to make this work.</div>





<div><br></div><div>+/-1 on changing the region constructor?</div><div><br></div><div>Cameron<br><div><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></div></div>
<br></div></div>_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org" target="_blank">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>
<br>_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org" target="_blank">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><br></div>
</div></div><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><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>