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

[CV] Request new entry: base peak intensity maximum until RT #247

Open
julianu opened this issue Dec 3, 2024 · 4 comments
Open

[CV] Request new entry: base peak intensity maximum until RT #247

julianu opened this issue Dec 3, 2024 · 4 comments
Labels

Comments

@julianu
Copy link
Collaborator

julianu commented Dec 3, 2024

What is the QC term's name?

base peak intensity maximum until RT

Briefly describe the QC term.

this term is similar to #246 but only takes the BPs into account until a given retention time.
This could maybe be modeled using a tuple or table specifying the RT and the maximum BP until this RT
Also #246 could be modelled using e.g. "-1" as RT to specify "global maximum BP"

What is the QC term's unit?

intensity / arbitrary and time (in seconds)

Value type

MS:4000004 ! n-tuple

Describe any additional information.

This metric is similar to #246 , but can cut away a very high gradient change in the end of an MS run.

@bittremieux
Copy link
Collaborator

bittremieux commented Dec 13, 2024

A table rather than an n-tuple I think, because n-tuples should contain similar values, whereas tables have different columns (in this case, base peak and retention time maximum). Does it make sense to generalize to an RT interval, e.g. both an RT minimum and maximum? Or to predefine RT intervals, such as in the QuaMeter quartile metrics?

@julianu
Copy link
Collaborator Author

julianu commented Jan 10, 2025

Generalizing for a RT interval makes perfectly sense.
The idea behind this was, that we look at the BP in a certain RT range, which does not include the very strong elution step / increase of gradient common at the end of a run's RT.

@julianu
Copy link
Collaborator Author

julianu commented Jan 16, 2025

I generalized it a bit more, so that the table may contain several rows.

Recommended term:

[Term]
id: MS:4000xyz
name: maximum base peak intensity in RT range
def: "This metric reflects the maximum base peak intensity (MS:1000505) of all spectra in a single run in given retention time windows. The retention time windows are specified by the 2nd column defining the start time in seconds and the 3rd column defining the end time in seconds. To only define a starting time, use -1 in the 3rd column." [PSI:MS]
is_a: MS:4000005 ! table
relationship: has_column MS:1000505 ! base peak intensity
relationship: has_column UO:0000010 ! second
relationship: has_column UO:0000010 ! second
relationship: has_metric_category MS:4000009 ! ID free metric
relationship: has_metric_category MS:4000012 ! single run based metric
relationship: has_metric_category MS:4000015 ! multiple spectra based metric

@bittremieux
Copy link
Collaborator

  • I'd maybe write "retention time" in full in the term's name.
  • Discussion point: Should the time-based columns maybe be MS:1000016 ! scan start time instead?

Otherwise it looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants