Skip to content

Conversation

@keesh0
Copy link

@keesh0 keesh0 commented Dec 10, 2024

Motivation:
Helps ascertain if there are disproportionately more MS2 triggers which in turn inflates the spectral counts for that protein.
This alters the ability to assess peak area and contributes to more variability in the assays.
We would hope for more peaks to be centered near the 1-line (line slope=1).

A convenient Jupyter notebook for example plotting is available from [email protected] .
Less changes would have been necessary if not for the near duplicate data structures (i.e. SlimCrawPeak,
CrawPeak, Peak, CrawdadPeak).

dpsmca and others added 30 commits May 22, 2022 19:12
…s it anyway and this will make tab completion easier on Linux and MacOS
@keesh0
Copy link
Author

keesh0 commented Jan 9, 2025

Anything I can do to move this pull request forward?

@chambm
Copy link
Member

chambm commented Jan 30, 2025

Sorry for the big delay reviewing this. Please revert the whitespace changes to make the changes easier to review and while you're at it remove the // BEG and //END comments (if I want to know where some code came from I'll just use git blame). I'm not a big fan of the extra output file. If it's important that the existing output columns stay the same, you can enable the new columns with a command-line option. Other than that it looks fine, thanks!

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.

4 participants