Skip to content

FIX: Lo DE handle partial ASCs#2600

Merged
greglucas merged 1 commit intoIMAP-Science-Operations-Center:devfrom
greglucas:imap-lo-spin-data-missing
Jan 22, 2026
Merged

FIX: Lo DE handle partial ASCs#2600
greglucas merged 1 commit intoIMAP-Science-Operations-Center:devfrom
greglucas:imap-lo-spin-data-missing

Conversation

@greglucas
Copy link
Collaborator

The spin_cycle variable is an average value over an ASC (28 spins). This means that when we have a partial ASC, the spin_cycle value could be in a "bad" spin which was reported as 0 in the ASC for spin_start_sec. To avoid this, we can use the average spin duration of the ASC and calculate the spin_start based on the start of the acquisition period of the ASC. This means we will always have a valid spin_start rather than 0 which caused SPICE lookup issues.

This gets l1b-de products being produced again and looks like it worked for a few quick old processing failures I tried locally.

The spin_cycle variable is an average value over an ASC (28 spins).
This means that when we have a partial ASC, the spin_cycle value
could be in a "bad" spin which was reported as 0 in the ASC
for spin_start_sec. To avoid this, we can use the average spin
duration of the ASC and calculate the spin_start based on the
start of the acquisition period of the ASC. This means we will
always have a valid spin_start rather than 0 which caused SPICE
lookup issues.
@greglucas greglucas added this to the January 2026 milestone Jan 21, 2026
@greglucas greglucas self-assigned this Jan 21, 2026
@greglucas greglucas added Ins: Lo Related to the IMAP-Lo instrument Level: L1 Level 1 processing labels Jan 21, 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.

LGTM

@greglucas greglucas merged commit f1bd87e into IMAP-Science-Operations-Center:dev Jan 22, 2026
14 checks passed
@greglucas greglucas deleted the imap-lo-spin-data-missing branch January 22, 2026 00:00
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 Level: L1 Level 1 processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants