Skip to content

Support new versioning scheme#137

Open
pleasant-menlo wants to merge 6 commits into
mainfrom
major-minor-version-format-simple
Open

Support new versioning scheme#137
pleasant-menlo wants to merge 6 commits into
mainfrom
major-minor-version-format-simple

Conversation

@pleasant-menlo

Copy link
Copy Markdown
Collaborator

Change Summary

Overview

Implemented handling of major version in IMAP L3 processing including the jobs that manage their own version numbers (cron jobs for GLOWS and maps).

The dependency JSON can now contain additional information about the requested version:

{
  "version": {
    "sci": {
      "major_version": 2,
      "minor_version": 1
    }
  },
  "dependency": [...]
}

We currently assume that version information will only be provided for a single descriptor, because our existing dependency configuration only specifies a single output for each job. This may need to be revisited if the dependency system later expects several outputs for a single L3 job.

The parallel change to imap_processing was made in IMAP-Science-Operations-Center/imap_processing#3313.

See IMAP-Science-Operations-Center/sds-data-manager#1397 for the top-level issue.

Closes #118.

File changes

Version objects are now created from the dependency JSON when they contain version numbers and otherwise fall back to creating version objects from the command line arguments.

Testing

Invoking any job from the command line using a dependency JSON that includes the version numbers should result in an output with the corresponding major and minor version numbers.

@pleasant-menlo pleasant-menlo changed the title Major minor version format simple Support new versioning scheme Jun 26, 2026
@jtniehof

Copy link
Copy Markdown
Collaborator

@bryan-harter @tech3371 @vineetbansal , can one of you take a look at this?

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.

Update L3 repo to take in new science file version - Card 3295

2 participants