Skip to content

MNT: Separate out Lo l1b dataset logic into individual functions#2601

Merged
greglucas merged 1 commit intoIMAP-Science-Operations-Center:devfrom
greglucas:lo-l1b-refactor
Jan 23, 2026
Merged

MNT: Separate out Lo l1b dataset logic into individual functions#2601
greglucas merged 1 commit intoIMAP-Science-Operations-Center:devfrom
greglucas:lo-l1b-refactor

Conversation

@greglucas
Copy link
Collaborator

The l1b dataset logic was within the if-blocks based on what dependency data was passed to the routine. This can be determined based on the descriptor requested and it can also be broken up into individual helper routines where the main code logic can be more easily identified and separated.

The l1b dataset logic was within the if-blocks based on what
dependency data was passed to the routine. This can be determined
based on the descriptor requested and it can also be broken up
into individual helper routines where the main code logic can be
more easily identified and separated.
@greglucas greglucas added this to the January 2026 milestone Jan 22, 2026
@greglucas greglucas self-assigned this Jan 22, 2026
@greglucas greglucas added the Ins: Lo Related to the IMAP-Lo instrument label Jan 22, 2026
Copy link
Contributor

@subagonsouth subagonsouth left a comment

Choose a reason for hiding this comment

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

Excellent!

@greglucas greglucas merged commit a0554e2 into IMAP-Science-Operations-Center:dev Jan 23, 2026
14 checks passed
@greglucas greglucas deleted the lo-l1b-refactor branch January 23, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ins: Lo Related to the IMAP-Lo instrument

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants