-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge BLAST XML tool #105
base: master
Are you sure you want to change the base?
Merge BLAST XML tool #105
Conversation
BTW this would be made obsolete by galaxyproject/galaxy#5464 if it was achieved. |
Assuming we later have (We would not want to try merging a mixture unless someone produces a robust script to interconvert from one format to the other). |
If |
If I read the Travis logs right the error isn't coming from this tool. Am I right? |
Grr. Another new TravisCI failure, but not your fault:
Update: Cross reference d31e21d and https://twitter.com/pjacock/status/998941086168076288 |
The error I am expecting to see is against the Galaxy master branch only, |
@peterjc sadly the test against |
Hmm. At the very least there were SQLite locking issues, which seems to be an intermittent problem. I've restarted that test on TravisCI, fingers crossed it gets further. |
The dev branch failed again on re-testing. In good news, the yarn version issue problem has been fixed on the Galaxy master branch, so I've asked TravisCI to retest that. |
As discussed on galaxyproject/tools-iuc#1893 this turns the merge code from the Galaxy BLAST XML datatype into a tool that can merge a list of datasets (or multiple individual datasets) into a single BLAST XML dataset.