<div dir="ltr"><div class="gmail_extra"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">​Hi Matt,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Yeah, I think I'll take you up on your offer to issue the PR.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Thanks,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Jon​</div><br><div class="gmail_quote">On Tue, Mar 8, 2016 at 3:45 PM,  <span dir="ltr"><<a href="mailto:yt-users-request@lists.spacepope.org" target="_blank">yt-users-request@lists.spacepope.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":196" class="a3s" style="overflow:hidden">Date: Tue, 8 Mar 2016 14:41:42 -0600<br>
From: Matthew Turk <<a href="mailto:matthewturk@gmail.com">matthewturk@gmail.com</a>><br>
To: Discussion of the yt analysis package<br>
        <<a href="mailto:yt-users@lists.spacepope.org">yt-users@lists.spacepope.org</a>><br>
Subject: Re: [yt-users] found the problem with non-square frb<br>
Message-ID:<br>
        <CALO3=5G0vR-=7G+KaQZo=<a href="mailto:Z5-7VoHngFZJvmbduensONdMcHbpQ@mail.gmail.com">Z5-7VoHngFZJvmbduensONdMcHbpQ@mail.gmail.com</a>><br>
Content-Type: text/plain; charset=UTF-8<br>
<br>
Hi Jon,<br>
<br>
That would be an awesome pull request, but I understand your<br>
reluctance.  I'm happy to issue it for you and credit you in the PR.<br>
:)<br>
<br>
-Matt<br>
<br>
On Tue, Mar 8, 2016 at 2:39 PM, Slavin, Jonathan<br>
<<a href="mailto:jslavin@cfa.harvard.edu">jslavin@cfa.harvard.edu</a>> wrote:<br>
> Hi all,<br>
><br>
> I just located the problem referred to in my earlier message.  There's a<br>
> very small bug in data_objects/data_containers.py<br>
> On lines 957 - 959 there is:<br>
>         elif iterable(height):<br>
>             h, u = height<br>
>             height = self.ds.quan(w, input_units = u)<br>
> Note the w in self.ds_quan(w, input_units = u).  Just replace that w with an<br>
> h and it works.  Should I file a pull request?  Seems a bit much for such a<br>
> minor bug, but I'm willing to do it.<br>
><br>
> Jon</div></blockquote></div><br><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">________________________________________________________<br>Jonathan D. Slavin                 Harvard-Smithsonian CfA<br><a href="mailto:jslavin@cfa.harvard.edu" target="_blank">jslavin@cfa.harvard.edu</a>       60 Garden Street, MS 83<br>phone: (617) 496-7981       Cambridge, MA 02138-1516<br>cell: (781) 363-0035             USA<br>________________________________________________________<br><br></div></div></div></div>
</div></div>