Skip to content

Add sdf format exporter#713

Merged
YukiMatsuzawa merged 9 commits intomasterfrom
add_sdf_format_exporter
Mar 10, 2026
Merged

Add sdf format exporter#713
YukiMatsuzawa merged 9 commits intomasterfrom
add_sdf_format_exporter

Conversation

@mikikot113
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 fixes and extends the SDF format exporter in SpectraExport.cs. Previously, the SDF exporter had incorrect field labels (InChIKey and SMILES were both labeled "FORMULA") and used "PRECURSOR MZ" instead of the standard "PRECURSOR M/Z". This PR corrects these labels and adds new metadata fields (EXACT MASS, RETENTION TIME, ONTOLOGY) to the SDF output.

Changes:

  • Fixed field labels: "PRECURSOR MZ" → "PRECURSOR M/Z", and corrected the duplicate "FORMULA" labels for InChIKey and SMILES to their proper names ("INCHIKEY", "SMILES")
  • Added new SDF metadata fields: EXACT MASS, RETENTION TIME, and ONTOLOGY in both WriteChromPeakFeatureInfoAsSdf overloads
  • Changed peak mass rounding precision from 5 to 4 decimal places in the AlignmentSpotProperty overload

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

mikikot113 and others added 2 commits March 10, 2026 10:50
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@YukiMatsuzawa YukiMatsuzawa merged commit db941b1 into master Mar 10, 2026
9 checks passed
@YukiMatsuzawa YukiMatsuzawa deleted the add_sdf_format_exporter branch March 10, 2026 02:59
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.

3 participants