-
Notifications
You must be signed in to change notification settings - Fork 5
Feature filter multiply labeled peptides #145
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
Changes from all commits
ed66faf
70b4051
63b3283
7edac98
556ab9d
01f0e1e
feee879
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -12,17 +12,23 @@ | |
| #' @param peptideSequenceColumn Name of the Spectronaut column that contains the | ||
| #' peptide sequence. Defaults to \code{"EG.ModifiedSequence"}. The value is | ||
| #' standardized internally (dots and spaces removed) before column lookup. | ||
| #' @param heavyLabels Character list identifying the heavy isotope labels as it | ||
| #' appears inside square brackets in the peptide sequence column, e.g. | ||
| #' \code{c("Lys6")} matches peptides containing \code{[Lys6]}. | ||
| #' \code{c("Lys6", "Arg10")} matches peptides containing either \code{[Lys6]} or \code{[Arg10]}. | ||
| #' Supports any novel label name reported by Spectronaut (e.g. \code{"Leu6"}, | ||
| #' \code{"Phe10"}). When provided, peptides are | ||
| #' classified as heavy (\code{IsotopeLabelType = "H"}), light | ||
| #' (\code{IsotopeLabelType = "L"}), or unlabeled | ||
| #' (\code{IsotopeLabelType = NA}) based on its labeled sequence. When | ||
| #' \code{NULL} (default) all peptides receive \code{IsotopeLabelType = "L"}. | ||
| #' Useful for protein turnover experiments. | ||
| #' @param heavyLabels Character vector naming each labeled residue and its | ||
| #' heavy label as they appear in the peptide sequence column: the | ||
| #' single-letter amino acid code, then the label in square brackets. For | ||
| #' example \code{"K[Lys6]"}, or \code{c("K[Lys6]", "R[Arg10]")} for a | ||
| #' double-label experiment. Any label name Spectronaut reports is accepted. | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: #!/bin/bash
set -eu
printf '%s\n' '--- target documentation ---'
cat -n R/converters_SpectronauttoMSstatsFormat.R | sed -n '1,45p'
printf '%s\n' '--- classification implementation ---'
cat -n R/clean_Spectronaut.R | sed -n '175,225p'
printf '%s\n' '--- heavyLabels definitions and callers ---'
rg -n -C 3 'heavyLabels|heavy_regex|grepl\\(' R tests inst | head -240Repository: Vitek-Lab/MSstatsConvert Length of output: 7924 🏁 Script executed: #!/bin/bash
set -eu
printf '%s\n' '--- bound helper implementations ---'
rg -n -C 8 'classifyIsotopeLabelType|filterMultiplyLabeledPeptides' R
printf '%s\n' '--- heavyLabels call sites and validation ---'
rg -n -C 5 'heavyLabels' R testsRepository: Vitek-Lab/MSstatsConvert Length of output: 13600 🏁 Script executed: #!/bin/bash
set -eu
cat -n R/utils_clean_features.R | sed -n '382,410p'
printf '%s\n' '--- exact helper documentation and tests ---'
cat -n R/utils_clean_features.R | sed -n '338,360p'
rg -n -C 6 'Spectronaut|Lys6|Arg10|IsotopeLabelType' tests | head -260Repository: Vitek-Lab/MSstatsConvert Length of output: 2631 Escape regex metacharacters in
🤖 Prompt for AI Agents |
||
| #' The residue letter is required, because it is what tells MSstats which | ||
| #' peptides could have carried a label. | ||
| #' | ||
| #' Peptides carrying the label are marked heavy | ||
| #' (\code{IsotopeLabelType = "H"}), those that could carry it but do not are | ||
| #' light (\code{"L"}), and those with no labelable residue are \code{NA}. A | ||
| #' peptide with two or more labelable residues (counted across all labels | ||
| #' supplied) may be only partially labeled, which the two-state turnover | ||
| #' model cannot represent, so it is dropped and the number removed is | ||
| #' reported. | ||
| #' | ||
| #' Defaults to \code{NULL}: turnover mode off, every peptide marked light. | ||
| #' @param excludedFromQuantificationFilter Remove rows with F.ExcludedFromQuantification=TRUE Default is TRUE. | ||
| #' @param filter_with_Qvalue FALSE(default) will not perform any filtering. TRUE will filter out the intensities that have greater than qvalue_cutoff in EG.Qvalue column. Those intensities will be replaced with zero and will be considered as censored missing values for imputation purpose. | ||
| #' @param qvalue_cutoff Cutoff for EG.Qvalue. default is 0.01. | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wondering if it should be this in the same spirit as the DIANN regex where here you prevent something like K[]] from being captured...?
\\[[^\\]]*\\]