<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jan 4, 2015 at 9:51 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>I advocate for having a wait time where no PRs get merged for a while before we release to sort out bugs.  I'm personally somewhat hesitant about using yt tip right now for production level analysis after all of those merges, many of which didn't appear to have much review.  It was my understanding that we were going to wait to let the dust settle on some things when everyone was back at their desks next week.</div></div></blockquote><div><br></div></span><div>I think that's an extremely unfair statement to make.  All of the PRs were reviewed, many were in the queue for weeks or months, and all of them pass the answer tests or include known fixes for previously incorrect results.</div></div></div></div></blockquote><div><br></div><div>Looking at the PRs that got merged last night (<a href="https://bitbucket.org/yt_analysis/yt/pull-requests?displaystatus=merged">https://bitbucket.org/yt_analysis/yt/pull-requests?displaystatus=merged</a>) most have 1-2 approvals, and one has zero approvals.  That's what I meant by not having much review.  I was under the impression that for substantial changes in code, we wanted to have 3 reviews/approvals.  Obviously, for minor bugfixes, this isn't necessary, but it seemed like some of the PRs were making substantial modifications to the codebase.  </div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>If you feel that the tip is unsafe, I would encourage you to write additional tests that stress it in your problem domain.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class=""><div dir="ltr"><div><br></div><div>Either way, I wanted to see what the status of things were with all of the recent activity, since I hadn't heard anything on the list.</div><span><font color="#888888"><div><br></div><div>Cameron</div><div><div><br></div>-- <br><div>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></font></span></div>
<br></span>_______________________________________________<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>
<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><div class="gmail_signature">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>