Skip to content
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

missing explaination of mikado.bed in tutorial #385

Closed
cooketho opened this issue Mar 6, 2021 · 2 comments · Fixed by #400
Closed

missing explaination of mikado.bed in tutorial #385

cooketho opened this issue Mar 6, 2021 · 2 comments · Fixed by #400

Comments

@cooketho
Copy link

cooketho commented Mar 6, 2021

Can you please help me understand something about the tutorial? I'm reading here:

https://mikado.readthedocs.io/en/latest/Tutorial/

and I see at the mikado serialize step there is a command which uses a file called mikado.bed:

mikado serialise --json-conf configuration.yaml --xml mikado.blast.xml.gz --orfs mikado.bed --blast_targets

but I can't seem to find any step where this file was created, or any previous reference to it in the tutorial. I see a reference to orfs.bed, which the tutorial indicates is from transdecoder. Is this a typo, and the command should have passed orfs.bed to --orfs? Or is this something else? Furthermore, it looks like the text of that command got cut off in the tutorial because --blast_targets needs an argument, but it doesn't have one.

Thanks!

@lucventurini
Copy link
Collaborator

Dear @cooketho

Thank you for pointing this out! We are reviewing the documentation these days and we'll make sure to explain this better.

Next Monday I will try to clarify it here as well.

Many thanks again for using our tool and for coming to us with this question, it helps us making Mikado ever better.

Kind regards

Luca Venturini

lucventurini added a commit that referenced this issue Mar 15, 2021
@lucventurini
Copy link
Collaborator

Dear @cooketho

Many apologies for the delay, we were swooped up by our quest to improve the code coverage for the suite.

I have update the documentation (see 8b63828) and we will bring it up live soon.

Kind regards,

lucventurini added a commit that referenced this issue Apr 1, 2021
…ue-395-pick-pad-crash' and 'issue-389-prepare-bed12-output' into version-2.2.4.

Fix #136 #395 #389 #397 #396 #385
@lucventurini lucventurini linked a pull request Apr 1, 2021 that will close this issue
lucventurini added a commit that referenced this issue Apr 6, 2021
Fix documentation and GHA tests.

Fix #385
Fix #389 
Fix #395 
Fix #396 
Fix #397
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants