
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">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">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">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></body></html>