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

Mag L1B/L1C cdf #623

Merged
merged 8 commits into from
Jun 14, 2024

Conversation

maxinelasp
Copy link
Contributor

Change Summary

This is the change to create MAG L1B and L1C output files. It does a first pass at MAG L1B processing as well, since it's very simple. Mag L1C processing is not complete.

Overview

  • Added MAG L1B processing first draft
  • Added MAG L1B and L1C attributes
  • Added MAG L1C basic output
  • Added L1B and L1C to cli.py

I have tested this against the CLI locally and it works great. I may need to update the SDS dependencies for L1C.

@maxinelasp maxinelasp requested review from a team, bourque, sdhoyt, greglucas, subagonsouth, tech3371, daralynnrhode, anamanica and vmartinez-cu and removed request for a team June 13, 2024 21:29
@maxinelasp maxinelasp self-assigned this Jun 13, 2024
Copy link
Contributor

@sdhoyt sdhoyt left a comment

Choose a reason for hiding this comment

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

Everything looks good to me

Copy link
Collaborator

@greglucas greglucas left a comment

Choose a reason for hiding this comment

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

Just some minor comments, looks good overall.

@maxinelasp maxinelasp merged commit a4c3549 into IMAP-Science-Operations-Center:dev Jun 14, 2024
17 checks passed
@maxinelasp maxinelasp deleted the mag_l1b_cdf branch November 20, 2024 22:46
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.

3 participants