<div dir="ltr"><div><div>Thanks for all those eyes on that pull request.<br><br></div>It would also be great if someone could look over Corentin's refactoring of the RAMSES frontend:<br><br><a href="https://github.com/yt-project/yt/pull/1641">https://github.com/yt-project/yt/pull/1641</a><br><br></div>It really just needs someone to give it one final look over and sanity check. It's already had two rounds of reviews. It's a big change though and I really want one more person to look over it carefully. You don't need to be a RAMSES expert.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 11, 2018 at 9:46 AM, Nathan Goldbaum <span dir="ltr"><<a href="mailto:nathan12343@gmail.com" target="_blank">nathan12343@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"><div><div><div>Hi all,<br><br></div>Any chance we can get express review for this pull request:<br><br><a href="https://github.com/yt-project/yt/pull/1661" target="_blank">https://github.com/yt-project/<wbr>yt/pull/1661</a><br><br></div>It makes the travis and appveyor tests pass on master and fixes issues related to numpy 1.14.<span class="HOEnZb"><font color="#888888"><br><br></font></span></div><span class="HOEnZb"><font color="#888888">-Nathan<br></font></span></div>
</blockquote></div><br></div>