<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi devs,<div class=""><br class=""></div><div class="">My apologies for making this request—I have some PRs that I would like reviewed quickly because they are in direct response to queries from users on the mailing list for my photon simulator analysis module. They fix some major bugs, and directly affect the analysis those users are doing now. </div><div class=""><br class=""></div><div class="">They are:</div><div class=""><br class=""></div><div class="">1790:</div><div class=""><br class=""></div><div class=""><a href="https://bitbucket.org/yt_analysis/yt/pull-requests/1790/bugfix-fix-issues-with-simput-files/" class="">https://bitbucket.org/yt_analysis/yt/pull-requests/1790/bugfix-fix-issues-with-simput-files/</a></div><div class=""><br class=""></div><div class="">and </div><div class=""><br class=""></div><div class="">1802:</div><div class=""><br class=""></div><div class=""><a href="https://bitbucket.org/yt_analysis/yt/pull-requests/1802/bugfix-fixing-bugs-and-big-speedups-for/" class="">https://bitbucket.org/yt_analysis/yt/pull-requests/1802/bugfix-fixing-bugs-and-big-speedups-for/</a></div><div class=""><br class=""></div><div class="">1790 has been passed by fido, but the tests for 1802 will be running soon. Could I get some eyes on these so we could perhaps get them merged in today? </div><div class=""><br class=""></div><div class="">I realize that reviewing these is sometimes a bit obtuse since they are rather domain-specific, so I thank you in advance for taking the time to look at them. I’m currently in the process of generating a comprehensive set of tests for the module so that fewer issues like these will crop up. </div><div class=""><br class=""></div><div class="">Best,</div><div class=""><br class=""></div><div class="">John</div></body></html>