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.

Sign up for GitHub

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

Jump to bottom

Add SelexION separation voltage #216

Merged

edeutsch merged 2 commits into master from edeutsch-separation-voltage

Oct 7, 2023

Contributor

edeutsch commented Oct 2, 2023

Add new term SelexION separation voltage based on documentation here:
https://sciex.com/content/dam/SCIEX/pdf/tech-notes/all/SelexION-Technology-Solution-to-Selectivity-Challenges.pdf


          Add SelexION separation voltage

33b4476

edeutsch requested review from mobiusklein and chambm

October 2, 2023 16:33

Contributor

mobiusklein commented Oct 6, 2023

Is this distinct from SelexION compensation voltage? https://www.ebi.ac.uk/ols4/ontologies/ms/classes/http%253A%252F%252Fpurl.obolibrary.org%252Fobo%252FMS_1003371?lang=en

Contributor Author

edeutsch commented Oct 6, 2023

I think yes, based on the documentation here: https://sciex.com/content/dam/SCIEX/pdf/tech-notes/all/SelexION-Technology-Solution-to-Selectivity-Challenges.pdf


          Update header

acfd4de

edeutsch enabled auto-merge

October 7, 2023 03:36

Contributor Author

edeutsch commented Oct 7, 2023

@mobiusklein I can't merge without an approval. Would you approve if you're happy with this?

edeutsch added this pull request to the merge queue

mobiusklein approved these changes

View reviewed changes

Merged via the queue into master with commit d1aacce

2 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet