Skip to content

Conversation

@bernt-matthias
Copy link
Contributor

@bernt-matthias bernt-matthias commented Nov 22, 2023

We just added this repo to the planemo monitor galaxyproject/planemo-monitor#43 and I noticed that conda struggles to find a blast+ package

@peterjc
Copy link
Owner

peterjc commented Nov 22, 2023

This rings a bell, I think some systems ended up with blast meaning legacy BLAST (e.g. Debiain), and others BLAST+ (e.g. conda).

I don't know why the linting is complaining about no .shed.yaml file, do you fancy writing one?

@bernt-matthias
Copy link
Contributor Author

no .shed.yaml file

True. So this thing never made it to the toolshed. The question is: should it?

@peterjc
Copy link
Owner

peterjc commented Nov 22, 2023

Good point. I never ended up using it personally (we have a local mirror of the NCBI databases on our cluster, updated monthly, and our Galaxy BLAST configuration just pointed at that), and am not in a position to even try it right now (we don't have a Galaxy server currently).

@peterjc
Copy link
Owner

peterjc commented Mar 26, 2025

Belatedly should we merge this now?

@bernt-matthias
Copy link
Contributor Author

Belatedly should we merge this now?

A few small changes would be needed (which I could do), but the biggest problem I have with the data manager is that it writes to blastdb in any case, but it should write to blastdb_p sometimes, isn't it? How abou the other datatables?

Should we add a select?

@peterjc
Copy link
Owner

peterjc commented Mar 26, 2025

I didn't notice that oversight.

[As an aside, I'm planning to ask the IUC to adopt this repository or at least the main BLAST wrappers later this year if I'm still not using Galaxy in my day job.]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants