Add statistical uncertainty pointers to SWE L2 data variables - #3376
Open
bryan-harter with Copilot wants to merge 2 commits into
Open
Add statistical uncertainty pointers to SWE L2 data variables#3376bryan-harter with Copilot wants to merge 2 commits into
bryan-harter with Copilot wants to merge 2 commits into
Conversation
Co-authored-by: bryan-harter <41062454+bryan-harter@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add SWE DELTA_* uncertainty pointers to metadata
Add statistical uncertainty pointers to SWE L2 data variables
Aug 10, 2026
bryan-harter
marked this pull request as ready for review
August 10, 2026 20:28
Member
|
I gave this ticket to AI, and it appears to have put things in the right place? 🤷 Anyone else with knowledge of SWE have insight? |
tech3371
approved these changes
Aug 13, 2026
tech3371
left a comment
tech3371
left a comment
Contributor
There was a problem hiding this comment.
yep, looks like it was that simple lol
Contributor
There was a problem hiding this comment.
I don't understand this file updates though :)
Contributor
|
Pinging @jtniehof and @pleasant-menlo for this change coming up. Hope this won't effect L3 pipeline :) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The SWE L2 CDF configuration lacked
DELTA_PLUS_VARandDELTA_MINUS_VARattributes for primary data arrays, which are necessary to link the original variables to their corresponding statistical uncertainties.Configuration Updates
flux: AddedDELTA_MINUS_VARandDELTA_PLUS_VARpointing toflux_stat_uncert.phase_space_density: AddedDELTA_MINUS_VARandDELTA_PLUS_VARpointing topsd_stat_uncert.