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

Add spectrum filter control to Import/Run Peptide Search wizard #3360

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chambm
Copy link
Member

@chambm chambm commented Feb 7, 2025

  • added spectrum filters control from Instrument tab to Transition Settings page of Import/Run Peptide Search wizard
  • updated TestDiaSearchStellarTutorialDraft to use spectrum filters to exclude alignment scans instead of using prefiltered mzML files
  • updated TestDiaSearchStellarTutorialDraft with more appropriate settings for ion trap data and pause for screenshot for TransitionSettingsPage
  • fixed SpectrumClassFilter.GetText() to separate terms by OR even if they captions
  • changed Edit Isolation Scheme dialog to allow deleting rows (enabled row headers) and, when importing windows from a file, automatically exclude windows that will be filtered out by the current spectrum filter
  • fixed audit logging of spectrum filter

Here's what audit logging looks like now:
Screenshot 2025-02-07 104911

…ttings page of Import/Run Peptide Search wizard

- updated TestDiaSearchStellarTutorialDraft to use spectrum filters to exclude alignment scans instead of using prefiltered mzML files
- updated TestDiaSearchStellarTutorialDraft with more appropriate settings for ion trap data and pause for screenshot for TransitionSettingsPage
- fixed SpectrumClassFilter.GetText() to separate terms by OR even if they captions
- changed Edit Isolation Scheme dialog to allow deleting rows (enabled row headers) and, when importing windows from a file, automatically exclude windows that will be filtered out by the current spectrum filter
@chambm chambm requested a review from nickshulman February 7, 2025 17:14
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.

1 participant