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

Possible inconsistencies #125

Open
robinschmid opened this issue Aug 18, 2021 · 2 comments
Open

Possible inconsistencies #125

robinschmid opened this issue Aug 18, 2021 · 2 comments

Comments

@robinschmid
Copy link
Collaborator

1. MS1MZ and MS2PROD do the same filtering on different MS levels.

Maybe we can use a similar names then to describe signals?
Options would include:

MS1MZ=200 / MS2MZ / MS3MZ
MZ=200:MS2
MZ=200:MSLEVEL:2

Maybe with convention over configuration we could even have something like:
QUERY scaninfo(MS2DATA) WHERE mz=200
without specifying the MS level it is inferred from the query statement

2. Use of scaninfo() with MS1DATA and MS2DATA

The transformation with scaninfo() does not feel like the rest of the language.

Maybe we can just add scandata to the modification functions (replacing MS1DATA and MS2DATA) and add the MSLEVEL=2 as a condition

QUERY **scandata** | scaninfo | scansum | scannum WHERE **MSLEVEL=2** AND ....

Feels a bit more natural. The MSLEVEL is a scan property like the retention time etc

@mwang87
Copy link
Owner

mwang87 commented Aug 19, 2021

You won't be able to put the MSLEVEL in the conditions, you might be able to put it in the filter section. Since you want to match MS1 and MS2 in the conditions, but only get the results from the MS2.

You could do this:

QUERY scandata WHERE MS2PROD=100 AND MS1MZ=500 AND MS2PREC=500 FILTER MSLEVEL=2

@robinschmid
Copy link
Collaborator Author

I like the
QUERY scandata WHERE .... format.

The end result of scaninfo(MS2DATA) acts like a condition - in the end you will remove all MS1 scans from the results (similar to conditions). The only thing that you need for the MS1MZ to work is the relationship between MS2 scans and their preceding MS1 scans.

Something like ms2scan.get_preceding_ms1()

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