Skip to content

Populate map variable CATDESC based on descriptor.#5

Merged
pleasant-menlo merged 3 commits intoIMAP-Science-Operations-Center:mainfrom
jtniehof:map_catdesc
Mar 6, 2026
Merged

Populate map variable CATDESC based on descriptor.#5
pleasant-menlo merged 3 commits intoIMAP-Science-Operations-Center:mainfrom
jtniehof:map_catdesc

Conversation

@jtniehof
Copy link
Collaborator

@jtniehof jtniehof commented Feb 17, 2026

This PR depends on IMAP-Science-Operations-Center/imap_processing#2712 and the rationale is explained there. This applies the same change to L3 code.

It's not particularly clean because the variable attributes are made fairly deep in the CDF writing code, so this really is triggering off variable name in a way that might be oversensitive.

@jtniehof
Copy link
Collaborator Author

Closing and reopening to try and trigger a resync. The new code should be a bit more intelligent about how it responds to variable names.

@jtniehof jtniehof closed this Feb 18, 2026
@jtniehof jtniehof reopened this Feb 18, 2026
@jtniehof jtniehof marked this pull request as ready for review February 24, 2026 16:38
@jtniehof
Copy link
Collaborator Author

jtniehof commented Mar 5, 2026

Since there's been a release of the imap_processing repo this is probably mergeable now.

@jtniehof jtniehof added this to IMAP Mar 5, 2026
@jtniehof jtniehof moved this to PR Open in IMAP Mar 5, 2026
@pleasant-menlo pleasant-menlo merged commit a6118fd into IMAP-Science-Operations-Center:main Mar 6, 2026
1 check passed
@github-project-automation github-project-automation bot moved this from PR Open to Done in IMAP Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants