Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated IDEX dependency to distinguish l1a and l1b #335

Merged

Conversation

bourque
Copy link
Collaborator

@bourque bourque commented Sep 16, 2024

This PR updates the IDEX dependencies to distinguish L1a and L1b processing levels (see IMAP-Science-Operations-Center/imap_processing#831)

Closes #334

@bourque bourque self-assigned this Sep 16, 2024
@@ -67,7 +67,8 @@ hit, l1a, hk, hit, l1b, hk, HARD, DOWNSTREAM

# <---- IDEX Dependencies ---->

idex, l0, raw, idex, l1, sci, HARD, DOWNSTREAM
idex, l0, raw, idex, l1a, sci, HARD, DOWNSTREAM
idex, l1a, raw, idex, l1b, sci, HARD, DOWNSTREAM
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is sci the name of the file descriptor that IDEX wants for all their L1 products?

Suggested change
idex, l1a, raw, idex, l1b, sci, HARD, DOWNSTREAM
idex, l1a, sci, idex, l1b, sci, HARD, DOWNSTREAM

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oops, good catch!

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 what Greg commented!

@bourque bourque requested a review from greglucas September 17, 2024 18:48
@greglucas greglucas merged commit 35634cc into IMAP-Science-Operations-Center:dev Sep 17, 2024
2 checks passed
@bourque bourque deleted the update-idex-dependencies branch September 17, 2024 18:50
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 IDEX dependencies with new L1a/L1b distinction
3 participants