Skip to content

Add GCMS normalized feature export#750

Merged
YukiMatsuzawa merged 1 commit into
masterfrom
copilot/yukimatsuzawa-gcms-normalized-exports
Jul 16, 2026
Merged

Add GCMS normalized feature export#750
YukiMatsuzawa merged 1 commit into
masterfrom
copilot/yukimatsuzawa-gcms-normalized-exports

Conversation

@YukiMatsuzawa

Copy link
Copy Markdown
Contributor

GCMS exports now include normalized feature output after normalization is available, so downstream analysis can consume the normalized peak data directly from the alignment export flow.

This change wires the normalized export options into the GCMS method model and keeps them gated on the existing normalized-data availability check, matching the behavior already used by other export surfaces.

Notes

  • Normalized height and normalized area are enabled only when normalized data can be exported.
  • No other export behavior is changed.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR enables GCMS alignment exports to include normalized peak height/area outputs when normalized data is available, aligning GCMS export capabilities with other method models that already support normalized exports.

Changes:

  • Adds “Normalized data (Height)” and “Normalized data (Area)” export types to the GCMS export quant type list.
  • Gates the normalized export types behind the existing CanExportNormalizedData(...) availability check via isNormalized.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@YukiMatsuzawa
YukiMatsuzawa merged commit 88dd264 into master Jul 16, 2026
10 checks passed
@YukiMatsuzawa
YukiMatsuzawa deleted the copilot/yukimatsuzawa-gcms-normalized-exports branch July 16, 2026 02:40
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.

2 participants