Skip to content

IDEX add MODS attribute for major version updates - #3487

Merged
lacoak21 merged 2 commits into
IMAP-Science-Operations-Center:devfrom
lacoak21:idex_major_update_mods_attr
Sep 24, 2026
Merged

lacoak21 merged 2 commits into
IMAP-Science-Operations-Center:devfrom
lacoak21:idex_major_update_mods_attr

Conversation

@lacoak21

@lacoak21 lacoak21 commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Change Summary

Overview

Add MODS attributes to global products.

@aldo9253 provided these!

@lacoak21 lacoak21 added this to the September 2026 milestone Sep 22, 2026
@lacoak21 lacoak21 self-assigned this Sep 22, 2026
@lacoak21 lacoak21 added this to IMAP Sep 22, 2026
Comment thread imap_processing/cdf/config/imap_idex_global_cdf_attrs.yaml
Comment thread imap_processing/cdf/config/imap_idex_global_cdf_attrs.yaml

@tmplummer tmplummer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

I'm wondering if we have any documentation for bumping major versions. If so, we should add updating product MODS attributes to the checklist. If not, we should probably write an issue to add some.

Comment thread imap_processing/cdf/config/imap_idex_global_cdf_attrs.yaml
@lacoak21

Copy link
Copy Markdown
Contributor Author

Hm yeah reading the docs again it seems like it might want a full history

https://istp-metadata.readthedocs.io/en/v1.0.0/source/03_metadata-global-attributes.html#mods:~:text=but%20each%20version,to%20the%20dataset.

@lacoak21
lacoak21 merged commit dceb4e9 into IMAP-Science-Operations-Center:dev Sep 24, 2026
22 checks passed
@github-project-automation github-project-automation Bot moved this to Done in IMAP Sep 24, 2026
@lacoak21
lacoak21 deleted the idex_major_update_mods_attr branch September 24, 2026 14:56
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