<div dir="ltr">+1; <div><br></div><div>I think it would be good to continue having individual api.py files for each of the analysis modules so that one can be sure to load all of the necessary functionality to run one analysis module at a time, but I don't think we need to have all of the analysis modules (which provide lots of disparate functionality) artificially wrapped into a single api as is currently done.  It has been the source of problems in the past (e.g if one module is broken, you cannot load any of them when loading from the general api).</div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 11, 2014 at 9:17 AM, John ZuHone <span dir="ltr"><<a href="mailto:jzuhone@gmail.com" target="_blank">jzuhone@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 style="word-wrap:break-word">Hi all,<div><br></div><div>I've issued a PR with some potentially controversial changes:</div>

<div><br></div><div><a href="https://bitbucket.org/yt_analysis/yt/pull-request/1009/removing-analysis-modules-as-per-trello/diff#comment-2383950" target="_blank">https://bitbucket.org/yt_analysis/yt/pull-request/1009/removing-analysis-modules-as-per-trello/diff#comment-2383950</a></div>

<div><br></div><div>which removes a few analysis modules as per this Trello card:</div><div><br></div><div><a href="https://trello.com/c/tJuM4kkk/23-remove-analysis-modules-as-needed" target="_blank">https://trello.com/c/tJuM4kkk/23-remove-analysis-modules-as-needed</a></div>

<div><br></div><div>Britton has also suggested that we do away with yt.analysis_modules.api entirely, as importing it imports a whole bunch of stuff you don't need, especially when you only need one module at a time (which is usually the case). </div>

<div><br></div><div>I'm +1 on this, but wanted to get others' takes. </div><div><br></div><div>Best,</div><div><br></div><div>John</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>