Skip to content

Correct blast#39

Open
cecilpert wants to merge 4 commits intopveber:masterfrom
cecilpert:correct_blast
Open

Correct blast#39
cecilpert wants to merge 4 commits intopveber:masterfrom
cecilpert:correct_blast

Conversation

@cecilpert
Copy link
Contributor

New PR for blast.ml, with modifications.

…an unique prefix name for results and recover them more easily
…to have an unique prefix name for results and recover them more easily"

This reverts commit 1f01944.
cmd "blastp" ~env [
opt "-db" dep db // db_name ;
opt "-query" dep query ;
opt "-out" ident (dest // results) ;
Copy link
Contributor Author

@cecilpert cecilpert Dec 6, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I just try :
opt "-db" dep db ;
opt "-out" ident dest ;
I have an error message like : Command line argument error: Argument "out". File is not accessible: /bistro/dest. I think blastp needs a file name and not just a directory as output.
I have the same error if I remove mkdir-p dest

@cecilpert
Copy link
Contributor Author

I forgot to add blast.mli, I'll write it later.

@pveber pveber force-pushed the master branch 3 times, most recently from fa83704 to 49c7a0e Compare July 6, 2022 05:58
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.

1 participant