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

Feature Request: check for gencode while creating index #366

Closed
alexvpickering opened this issue May 27, 2019 · 2 comments
Closed

Feature Request: check for gencode while creating index #366

alexvpickering opened this issue May 27, 2019 · 2 comments
Assignees
Labels
alevin issue is primarily related to alevin fixed in develop this bug has been fixed in develop and the issue will be closed when merged into master

Comments

@alexvpickering
Copy link
Contributor

This came up in #336

When building a gencode-based index with salmon index without the --gencode flag, the index that is generated won't work when subsequently running e.g. salmon alevin. This can be a pain to figure out the mistake (missing --gencode flag).

Perhaps some sort of quick grep of the index file name for 'gencode' or within the file for multiple '|'s (used for gencode field separator but not ensembl) should be done to check if it's gencode followed by a warning/error if the --gencode flag is missing? Another possibility could be to just auto-detect if the index is gencode using the above tests, removing the need for the --gencode flag. Thank you

@k3yavi k3yavi self-assigned this May 30, 2019
@k3yavi k3yavi added the alevin issue is primarily related to alevin label May 30, 2019
@k3yavi
Copy link
Member

k3yavi commented May 30, 2019

Thanks again @alexvpickering .
We have fixed this in the develop branch and it will be part of the salmon from the next release which we plan to release very soon.

@k3yavi k3yavi added the fixed in develop this bug has been fixed in develop and the issue will be closed when merged into master label May 30, 2019
@rob-p
Copy link
Collaborator

rob-p commented Jun 1, 2019

Now (since v0.14.0), the index command will issue a warning if you don't pass --gencode but the fasta being indexed looks to be gencode.

@rob-p rob-p closed this as completed Jun 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alevin issue is primarily related to alevin fixed in develop this bug has been fixed in develop and the issue will be closed when merged into master
Projects
None yet
Development

No branches or pull requests

3 participants