On Wed, Dec 4, 2013 at 4:21 PM, Peter Cock <p.j.a.cock@googlemail.com> wrote:
On Mon, Dec 2, 2013 at 3:37 PM, Peter Cock <p.j.a.cock@googlemail.com> wrote:
On Sun, Dec 1, 2013 at 7:10 PM, Björn Grüning <bjoern.gruening@pharmazie.uni-freiburg.de> wrote:
Great. I'll probably merge the c25 branch and push that to the Test Tool Shed next week (adding the descriptions as a new 25th column to the default extended tabular output).
+1
Done, see this and the preceding commits: https://github.com/peterjc/galaxy_blast/commit/eb1e522a864e5274a2d274a49fcb1...
And the Test Tool Shed repository has been updated: http://testtoolshed.g2.bx.psu.edu/view/peterjc/ncbi_blast_plus/8f9023b30384
We'll also be running this update locally for a little more testing, but I'd appreciate some of you also testing this, e.g. via the Test Tool Shed on your own local (development) Galaxy instances.
If there are no problems, that can probably go out to the main Tool Shed by the end of the week.
I pushed another update to the Test Tool Shed addressing a slight change in the autogenerated IDs used in the BLAST XML output, and an overlooked mention of 24 columns:
http://testtoolshed.g2.bx.psu.edu/view/peterjc/ncbi_blast_plus/72170c3f515a
The functional tests are passing on the Test Tool Shed, so before I push this to the main Tool Shed, any other comments?
Feedback is still welcome, but I have now pushed this to the main Tool Shed as v0.0.22 of the BLAST+ wrappers, which targets BLAST+ 2.2.28: http://toolshed.g2.bx.psu.edu/view/devteam/ncbi_blast_plus/4c4a0da938ff (I realise now this ended up skipping v0.0.21 which updated the dependencies to wrap BLAST+ 2.2.27) Maybe with the next set of changes like pick-you-own columns I will bump the minor version number and call this v0.1.0 to avoid any possibly confusion between the wrapper version and the underlying BLAST+ version - good plan? After all, it will cause some slight breakage if trying to re-run old jobs due to the introduction of a conditional block for the output format. Peter