Skip to content

CODICE l2 metadata updates#3310

Merged
lacoak21 merged 1 commit into
IMAP-Science-Operations-Center:devfrom
lacoak21:fix_codice_metadata_for_release_1
Jun 25, 2026
Merged

CODICE l2 metadata updates#3310
lacoak21 merged 1 commit into
IMAP-Science-Operations-Center:devfrom
lacoak21:fix_codice_metadata_for_release_1

Conversation

@lacoak21

@lacoak21 lacoak21 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Change Summary

closes #3311

Overview

These changes were outlined by Jon and the CoDICE team.

  1. Fix DISPLAY_TYPE metadata
    • Change DISPLAY_TYPE from time_series to spectrogram for all CoDICE L2 variables that are 2-dimensional. Currently some files (e.g., hi-omni) already do this correctly; others (e.g., lo files) still say time_series, which makes CAVA plot them as line plots instead of spectrograms. Straightforward, mechanical fix.
  2. Fix uncertainty variable handling
    • Uncertainties are currently defined as standalone/"free-floating" data variables, which makes them show up as separate plottable variables in CAVA (confusing). Instead, each uncertainty variable should be:
      • Marked VAR_TYPE: support_data (not data)
      • Attached to its parent variable via DELTA_PLUS_VAR / DELTA_MINUS_VAR pointing to the uncertainty variable name (e.g., unc_{species}) hi-omni already follows this pattern; lo-species and likely others need to be updated to match.

File changes

Update cdf attribute yaml files

@jtniehof

Copy link
Copy Markdown
Collaborator

#3293 might have fixed the RtD build

@jtniehof jtniehof left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. I haven't thrown test files at CAVA or anything, but that's consistent with metadata that plots successfully.

@davidt0x davidt0x left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@lacoak21 lacoak21 force-pushed the fix_codice_metadata_for_release_1 branch from 124059d to 4e87277 Compare June 25, 2026 13:59
@lacoak21 lacoak21 merged commit cc5f479 into IMAP-Science-Operations-Center:dev Jun 25, 2026
14 checks passed
@lacoak21 lacoak21 deleted the fix_codice_metadata_for_release_1 branch June 25, 2026 14:33
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.

Fix CoDICE metadata l2

3 participants