<html><head></head><body><div style="font-family:Helvetica Neue, Helvetica, Arial, sans-serif;font-size:10px;"><div style="font-family:Helvetica Neue, Helvetica, Arial, sans-serif;font-size:10px;"><div></div>
<div>Hi Peter,</div><div><br></div><div>I don't think we need to reintroduce the old API; the benefit of doing so for any users who may have used this function does not weigh up against the added complexity. Anyway the version() function was more for internal use and debugging, and it's easy to fix for users who use this function.<br></div><div><br></div><div>Best,</div><div>-Michiel<br></div><div><br></div>
</div><div id="yahoo_quoted_9883318797" class="yahoo_quoted">
<div style="font-family:'Helvetica Neue', Helvetica, Arial, sans-serif;font-size:13px;color:#26282a;">
<div>
On Saturday, June 23, 2018, 1:11:48 AM GMT+9, Peter Cock <p.j.a.cock@googlemail.com> wrote:
</div>
<div><br></div>
<div><br></div>
<div><div dir="ltr">On Fri, Jun 22, 2018 at 4:49 PM, Peter Cock <<a shape="rect" ymailto="mailto:p.j.a.cock@googlemail.com" href="mailto:p.j.a.cock@googlemail.com">p.j.a.cock@googlemail.com</a>> wrote:<br clear="none">> I'm part way though the release now, and for future reference<br clear="none">> I think I would now be able to simplify the build instructions<br clear="none">> as well...<br clear="none"><br clear="none">I'd normally ask the developer-only list, but since we merged<br clear="none">the mailing lists, if anyone wants to do any last minute local<br clear="none">testing before things get up to PyPI:<br clear="none"><br clear="none"><a shape="rect" href="http://biopython.org/DIST/biopython-1.72.tar.gz" target="_blank">http://biopython.org/DIST/biopython-1.72.tar.gz</a><br clear="none"><a shape="rect" href="http://biopython.org/DIST/biopython-1.72.zip" target="_blank">http://biopython.org/DIST/biopython-1.72.zip</a><br clear="none"><br clear="none">(There may still be some strange locale specific encoding issues<br clear="none">with the test suite which our current developers are not<br clear="none">running into on their own machines, there are some open issues<br clear="none">on GitHub about this - we need help here.)<br clear="none"><br clear="none">> The first hurdle is with the wheel building, it looks like I need<br clear="none">> to debug this cryptic error:<br clear="none">><br clear="none">> Traceback (most recent call last):<br clear="none">> File "<string>", line 1, in <module><br clear="none">> AttributeError: 'module' object has no attribute 'version'<br clear="none">><br clear="none">> <a shape="rect" href="https://travis-ci.org/biopython/biopython-wheels/builds/395521793" target="_blank">https://travis-ci.org/biopython/biopython-wheels/builds/395521793</a><br clear="none"><br clear="none">That seems easy to avoid, but suggests an accidental API<br clear="none">change to Bio.Cluster was introduced:<br clear="none"><br clear="none"><a shape="rect" href="https://github.com/biopython/biopython-wheels/commit/4018334b038278e8da3522039584e57dd361f135" target="_blank">https://github.com/biopython/biopython-wheels/commit/4018334b038278e8da3522039584e57dd361f135</a><br clear="none"><br clear="none">Michiel, is it worth re-creating the old API now, something<br clear="none">like this in Bio/Cluster/__init__.py perhaps?<br clear="none"><br clear="none">def version():<br clear="none"> return __version__<div class="yqt8475836347" id="yqtfd08798"><br clear="none"><br clear="none">Peter<br clear="none"></div></div></div>
</div>
</div></div></body></html>