-
Notifications
You must be signed in to change notification settings - Fork 13
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
Separate IDEX L1 into L1a and L1b #831
Separate IDEX L1 into L1a and L1b #831
Conversation
Closes #832 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, this is a nicer organization.
@@ -21,7 +21,7 @@ The general filename convention is as follows:: | |||
* CoDICE: ``l0``, ``l1a``, ``l1b``, ``l2``, ``l3`` | |||
* GLOWS: ``l0``, ``l1a``, ``l1b``, ``l2``, ``l3a``, ``l3b``, ``l3c``, ``l3d`` | |||
* HIT: ``l0``, ``l1a``, ``l1b``, ``l2``, ``l3`` | |||
* IDEX: ``l0``, ``l1``, ``l2``, ``l3`` | |||
* IDEX: ``l0``, ``l1a``, ``l1b``, ``l2``, ``l3`` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to update the sds-data-manager
repo with this as well for the dependency graph? If so, can you open an issue for that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah yes, good call.
The IDEX team informed us that they expect separate L1a and L1b data products (with the L1b data products being basically the same as the L1a data products, only converted from DN to pC). This PR makes various changes in order to distinguish between the two levels. Changes to create L1b products will be made in a separate PR.
I also took this PR as an opportunity to move the
idex_l0.py
andidex_l1[a|b].py
modules into the same directory. In my opinion, keeping L0 and L1 code in separate subpackages is overkill.