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

Adding the contam_ prefix doesn't work with an existing database. It only works when downloading the database using Philosopher #516

Open
lxavierneves opened this issue Dec 13, 2024 · 5 comments

Comments

@lxavierneves
Copy link

Hello,

I'm using Fragpipe v22.0 and loaded a uniprot reference proteome .fasta file then loaded into Fragpipe and clicked to add decoys and contaminants. However I noticed that the contaminants have not been marked with "contam_" in the .fasta file header, therefore have not been removed from my search results.

has this happened to other users?

many thanks,
Leandro

@fcyu
Copy link
Member

fcyu commented Dec 13, 2024

However I noticed that the contaminants have not been marked with "contam_" in the .fasta file header

It seems to be a bug in Philosopher. Could you share your fasta file?

Thanks,

Fengchao

@anesvi
Copy link

anesvi commented Dec 13, 2024

If you downloaded Human database, then contaminants that are human proteins would not be marked as contaminants, I believe. Maybe this will explain what you observed?

@lxavierneves
Copy link
Author

It's a hamster database and none of the contaminant entries had the contam_ prefix

File below
2024-12-13-decoys-contam-20241211_UP000189706_10036_onePTNperGene.fasta.zip

Many thanks

@lxavierneves
Copy link
Author

I've just realised that the original .fasta could be useful as well
20241211_UP000189706_10036_onePTNperGene.zip

@fcyu
Copy link
Member

fcyu commented Dec 13, 2024

Thanks for sharing the file. It remind me that adding the contam_ prefix only works when you download the database using Philosopher: #510 (comment)

Let me transfer this one to https://github.com/Nesvilab/philosopher/issues

Best,

Fengchao

@fcyu fcyu changed the title contaminants missing "contam_" parsing rule Adding the contam_ prefix doesn't work with an existing database. It only works when downloading the database using Philosopher Dec 13, 2024
@fcyu fcyu transferred this issue from Nesvilab/FragPipe Dec 13, 2024
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

No branches or pull requests

3 participants