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

combined_protein.tsv with empty sample columns #520

Open
loganjohnson0 opened this issue Jan 25, 2025 · 4 comments
Open

combined_protein.tsv with empty sample columns #520

loganjohnson0 opened this issue Jan 25, 2025 · 4 comments
Assignees

Comments

@loganjohnson0
Copy link

loganjohnson0 commented Jan 25, 2025

Hello FragPipe Team,

I have been analyzing a dataset with the TMT-bridge default workflow. Samples labeled with TMT10plex and a 131C bridge channel with 10 TMT runs. My question might be Philosopher related, so my apologises if that is the case.

FragPipe runs without error, but I noticed that the creation of the combined_protein.tsv file generates with each sample placed in a new column, yet no information exists in the rows of these sample columns. The indistinguishable proteins column is also empty, with the indistinguishable proteins placed in the first column of the sample columns (images below as an example).

Image Image

I am using an in-house generated contaminants file with contam_ labels in addition to a one-protein-per-gene UniProt Sus scrofa file and decoys added by FragPipe. I have some NCBI sequences appended, but they are annotated to prescribe to the requirements of Philosopher.

I have seen this in a few different datasets I've tried and wasn't sure if this was a feature or a bug. I'm generating an MSstats file with Philosopher and using that for my downstream analysis, and I haven't noticed an impact on any data. I am attaching my log file below and can share other files if it helps.

Thank you for the great software!

log_2025-01-25_13-48-47.txt

Version info:
FragPipe version 22.0
MSFragger version 4.1
IonQuant version 1.10.27
diaTracer version 1.1.5
Philosopher version 5.1.1
@fcyu fcyu self-assigned this Jan 25, 2025
@fcyu
Copy link
Member

fcyu commented Jan 25, 2025

The log file is not readable. Could you re-uploaded it again?

Thanks,

Fengchao

@loganjohnson0
Copy link
Author

Sorry, I realized that right after I submitted the issue. I have edited my original comment and have it below.

log_2025-01-25_13-48-47.txt

@fcyu
Copy link
Member

fcyu commented Jan 25, 2025

No worries. Yeah, it seems to be a bug in Philosopher. We actually don't use the combined_*.tsv tables much for the TMT data. The recommended tables are those in the tmt-report folder. And the bug should also not affect the MSstats files.

Best,

Fengchao

@loganjohnson0
Copy link
Author

Ya, I usually just quickly look at the combined_*.tsv files, but I figured I'd report it anyway. Thanks!

@fcyu fcyu transferred this issue from Nesvilab/TMT-Integrator Jan 26, 2025
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

2 participants