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

[CHANGE] ui option name for feature file boundaries #99

Merged
merged 4 commits into from
Jan 24, 2024

Conversation

singjc
Copy link
Collaborator

@singjc singjc commented Jan 24, 2024

Change peak picking option OSW-PyProphet to Feature File Boundaries

Contents (#99)

Other

  • Feature File Peak Boundaries for raw GUI workflow
  • Roestlab/massseer into patch/ui_rename_results_peak_boundaries

Uncategorised!

  • [CHANGE] ui option name for feature file boundaries
  • [RENAME] feature file peak pickign method

@singjc singjc closed this Jan 24, 2024
@singjc singjc reopened this Jan 24, 2024
@singjc singjc self-assigned this Jan 24, 2024
@singjc singjc added the refactor Refactoring code label Jan 24, 2024
@singjc singjc requested a review from jcharkow January 24, 2024 05:56
Copy link
Collaborator

@jcharkow jcharkow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes are good. However currently there is an issue in the raw peak GUI display failing to find features. Should try to address this in this PR as well because it concerns these. Unsure if we should use the plotChromatogram method I added to the loaders or decouple this feature reading from the peak picking server a bit because it is not doing peak picking just reading results and thus needs different arguments from the other peak pickers (e.g. transitionGroupFeatures).

massdash/server/PeakPickingServer.py Outdated Show resolved Hide resolved
@@ -35,7 +35,7 @@ def __init__(self, peak_picking_settings: PeakPickingUISettings, chrom_plot_sett

def perform_osw_pyprophet_peak_picking(self, xic_data: SqMassLoader, transition_list_ui: Literal['ExtractedIonChromatogramAnalysisUI', 'RawTargetedExtractionAnalysisUI']):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am having difficulties with this method in the GUI could you please look into this? I think that the loader object for the raw data is not currently being passed which causes loadTransitionGroupFeature to fail. Not sure what the best way to solve this without just adding more arguments

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're getting this error?

AttributeError: 'NoneType' object has no attribute 'loadTransitionGroupFeature'

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes thats the error. I think it is because there is no XIC data/loader object passed

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I passed the mzml loader object now instead to get the feature data from the results file. I think I initially used the sqMass workflow that's why it failed.

Feature file peak boundary rendering doesn't work for the raw GUI
workflow. Added the mzml loader object to 1Dplotting to get feature file
boundaries.
@singjc singjc requested a review from jcharkow January 24, 2024 18:59
Copy link
Collaborator

@jcharkow jcharkow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good for now however I do think this needs refactoring. I am thinking of separating out FeatureFile from the rest of peak picking because it is inherently different. Will create a separate branch for that though

@singjc singjc merged commit 36243d9 into dev Jan 24, 2024
10 checks passed
@singjc singjc deleted the patch/ui_rename_results_peak_boundaries branch January 24, 2024 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Refactoring code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants