Skip to content

I-ALiRT - bugfix for error propagation#3205

Merged
laspsandoval merged 1 commit into
IMAP-Science-Operations-Center:devfrom
laspsandoval:bugfix_for_packet_ingest
May 15, 2026
Merged

I-ALiRT - bugfix for error propagation#3205
laspsandoval merged 1 commit into
IMAP-Science-Operations-Center:devfrom
laspsandoval:bugfix_for_packet_ingest

Conversation

@laspsandoval
Copy link
Copy Markdown
Contributor

This pull request makes a targeted fix to the logic that handles year rollover detection for station data in the packets_created function. The change ensures that a new year is only detected when the previous day-of-year is late in the year (over 300) and the current day-of-year is early (under 30), preventing incorrect year increments for other cases.

  • Improved year rollover logic in packets_created (imap_processing/ialirt/calculate_ingest.py): Now only increments the year when transitioning from late in one year (DOY > 300) to early in the next (DOY < 30), reducing false positives.

@laspsandoval laspsandoval requested a review from lacoak21 May 15, 2026 18:00
@laspsandoval laspsandoval self-assigned this May 15, 2026
@laspsandoval laspsandoval added bug Something isn't working I-ALiRT labels May 15, 2026
Copy link
Copy Markdown
Contributor

@lacoak21 lacoak21 left a comment

Choose a reason for hiding this comment

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

Nice - looks good.

@laspsandoval laspsandoval merged commit 1541117 into IMAP-Science-Operations-Center:dev May 15, 2026
14 checks passed
@github-project-automation github-project-automation Bot moved this to Done in IMAP May 15, 2026
@laspsandoval laspsandoval deleted the bugfix_for_packet_ingest branch May 15, 2026 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working I-ALiRT

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants