Skip to content

Ultra ena rates l1b#2386

Merged
lacoak21 merged 2 commits intoIMAP-Science-Operations-Center:devfrom
lacoak21:ultra_ena_rates_l1b
Nov 6, 2025
Merged

Ultra ena rates l1b#2386
lacoak21 merged 2 commits intoIMAP-Science-Operations-Center:devfrom
lacoak21:ultra_ena_rates_l1b

Conversation

@lacoak21
Copy link
Copy Markdown
Contributor

@lacoak21 lacoak21 commented Nov 5, 2025

Change Summary

Overview

The spin edges were getting converted to uint16. This was causing the spins to roll over when it reached the max uint16 val. This was causing ena_rates to be an array of zeros because none of the spins were within the spin_edges.

unconverted spins [140309 140310 140311 ... 146033 146034 146035] converted spins [ 9237 9238 9239 ... 14961 14962 14963]

This was a remanent of when the spin_number was uint8. @laspsandoval helped me confirm this was the correct way to fix this issue.

Updated Files

  • imap_processing/ultra/l1b/ultra_l1b_culling.py
    • Dont convert spin numbers to uint16.

@lacoak21 lacoak21 added this to the November 2025 milestone Nov 5, 2025
@lacoak21 lacoak21 self-assigned this Nov 5, 2025
@lacoak21 lacoak21 added this to IMAP Nov 5, 2025
Copy link
Copy Markdown
Contributor

@laspsandoval laspsandoval left a comment

Choose a reason for hiding this comment

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

Thanks for doing this Luisa.

@lacoak21 lacoak21 merged commit 35cfdeb into IMAP-Science-Operations-Center:dev Nov 6, 2025
14 checks passed
@github-project-automation github-project-automation bot moved this to Done in IMAP Nov 6, 2025
@lacoak21 lacoak21 deleted the ultra_ena_rates_l1b branch November 6, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants