Skip to content

feat(public): display MIChI spectrum metadata on dataset page#1470

Open
vcnainala wants to merge 1 commit into
developmentfrom
feat/993-michi-dataset-page-display
Open

feat(public): display MIChI spectrum metadata on dataset page#1470
vcnainala wants to merge 1 commit into
developmentfrom
feat/993-michi-dataset-page-display

Conversation

@vcnainala

Copy link
Copy Markdown
Member

Summary

  • Adds michiSpectrumInfo.js to extract MIChI v1 fields that have an NMRium mapping (from spectra[0].info and spectra[0].filters)
  • Merges MIChI-labelled rows into the public dataset Spectrum info table ahead of remaining raw NMRium keys, avoiding duplicate solvent/nucleus/frequency rows
  • Links the section heading to the existing michiStandardsUrl Inertia prop

Implements the display scope of #993 (no DB columns, import command, or search filters in this PR).

Test plan

  • Open a public dataset with NMRium data and confirm MIChI rows appear first with units (MHz, s, K, °)
  • Confirm unmapped NMRium keys (e.g. pulse sequence, dimension) still appear after MIChI rows
  • Confirm datasets without filters still render info-only MIChI fields
  • Confirm the MIChI link opens the standards documentation

Extract MIChI v1 fields with NMRium mapping from spectrum info and
processing filters, and merge them into the public dataset spectrum
info table ahead of remaining raw NMRium keys.

Closes #993
@codecov

codecov Bot commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.59%. Comparing base (6ee0a22) to head (1563b5b).

Additional details and impacted files
@@              Coverage Diff               @@
##             development    #1470   +/-   ##
==============================================
  Coverage          76.59%   76.59%           
  Complexity          3593     3593           
==============================================
  Files                250      250           
  Lines              13521    13521           
==============================================
  Hits               10357    10357           
  Misses              3164     3164           
Flag Coverage Δ
unittests 76.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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