Done

On Wed, Nov 24, 2010 at 10:13 AM, Peter <peter@maubp.freeserve.co.uk> wrote:
Hi Kanwei,

Could you check and commit the following three changesets from
my new branch:

http://bitbucket.org/peterjc/galaxy-central/src/blastplus_nov24

i.e. these three changesets:

http://bitbucket.org/peterjc/galaxy-central/changeset/8f0e5a169630
http://bitbucket.org/peterjc/galaxy-central/changeset/72fab21f9469
http://bitbucket.org/peterjc/galaxy-central/changeset/72fab21f9469

This address a problem found earlier today by Alex Bossers (CC'd)
while testing the BLAST+ wrappers. The problem can occur when
when the query sequence is masked (with XXXX characters), this
broke my calculation of the identity and mismatch counts (failing
an assert statement I'd included as a sanity check). The new unit
test is a cut down version of his failing example.

Note that in order to run the unit tests, you will need to first
re-enable the tool in tools_conf.xml.sample - the converter
tests does not depend on the BLAST+ binaries being installed.

Thank you,

Peter