Skip to content

GLOWS L2 - Updated Delta +/- uncertainties attribute for photon flux#2731

Merged
sdhoyt merged 1 commit intoIMAP-Science-Operations-Center:devfrom
sdhoyt:glows-delta-var
Feb 19, 2026
Merged

GLOWS L2 - Updated Delta +/- uncertainties attribute for photon flux#2731
sdhoyt merged 1 commit intoIMAP-Science-Operations-Center:devfrom
sdhoyt:glows-delta-var

Conversation

@sdhoyt
Copy link
Contributor

@sdhoyt sdhoyt commented Feb 19, 2026

Change Summary

Added the DELTA_PLUS_VAR, DELTA_MINUS_VAR attributes to the yaml. The flux uncertainties are symmetric so the same value can be used for both the plus and the minus

Ran CLI to test CDF creation

Closes #2315

@sdhoyt sdhoyt self-assigned this Feb 19, 2026
@sdhoyt sdhoyt added Ins: GLOWS Related to the GLOWS instrument Level: L2 Level 2 processing CDF Related to CDF files labels Feb 19, 2026
@sdhoyt sdhoyt added this to IMAP Feb 19, 2026
Copy link
Contributor

@tech3371 tech3371 left a comment

Choose a reason for hiding this comment

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

looks good besides one minor question.

Comment on lines +411 to +412
DELTA_MINUS_VAR: flux_uncertainties
DELTA_PLUS_VAR: flux_uncertainties
Copy link
Contributor

Choose a reason for hiding this comment

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

Do other variables in this file need delta plus/minus too? Eg. flux_uncertainties or raw_uncertainties or others. If we don't know yet, it's ok. When you run this pass by Andriy, he has been good with identifying those needs. @jtniehof is also a good person to receive inputs like this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I'll look at that when I'm doing general metadata checks

@sdhoyt sdhoyt merged commit a0a1052 into IMAP-Science-Operations-Center:dev Feb 19, 2026
22 of 23 checks passed
@github-project-automation github-project-automation bot moved this to Done in IMAP Feb 19, 2026
@sdhoyt sdhoyt deleted the glows-delta-var branch February 19, 2026 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CDF Related to CDF files Ins: GLOWS Related to the GLOWS instrument Level: L2 Level 2 processing

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Add GLOWS DELTA_* uncertainty pointers to metadata GLOWS L2

2 participants

Comments