Skip to content

Commit 19f1d25

Browse files
authored
Report energy deltas in keV instead of eV (#2543)
* Report energy deltas in keV instead of eV * Copilot feedback changes
1 parent 4c3c135 commit 19f1d25

2 files changed

Lines changed: 17 additions & 5 deletions

File tree

imap_processing/lo/l2/lo_l2.py

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -762,12 +762,20 @@ def populate_geometric_factors(
762762
"geometric_factor_stat_uncert": f"GF_Trpl_{species.upper()}_unc",
763763
}
764764
if species == "h":
765-
# NOTE: From an e-mail from Nathan on 2025-09-11
766-
energy_delta_hires_values = [5.43, 10.02, 18.61, 33.31, 64.98, 131.64, 262.35]
767-
energy_delta_hithr_values = [8.81, 16.04, 28.50, 53.13, 105.60, 219.67, 413.60]
765+
# NOTE: From an e-mail from Nathan on 2025-09-11 (values converted to keV)
766+
energy_delta_hires_values = (
767+
np.array([5.43, 10.02, 18.61, 33.31, 64.98, 131.64, 262.35]) * 1e-3
768+
)
769+
energy_delta_hithr_values = (
770+
np.array([8.81, 16.04, 28.50, 53.13, 105.60, 219.67, 413.60]) * 1e-3
771+
)
768772
else: # species == "o"
769-
energy_delta_hires_values = [5.82, 11.10, 21.78, 41.47, 85.61, 180.67, 361.93]
770-
energy_delta_hithr_values = [9.45, 17.84, 33.51, 66.61, 139.95, 302.24, 569.48]
773+
energy_delta_hires_values = (
774+
np.array([5.82, 11.10, 21.78, 41.47, 85.61, 180.67, 361.93]) * 1e-3
775+
)
776+
energy_delta_hithr_values = (
777+
np.array([9.45, 17.84, 33.51, 66.61, 139.95, 302.24, 569.48]) * 1e-3
778+
)
771779

772780
# Get ESA mode from the map (assuming it's constant or we take the first)
773781
# TODO: Figure out how to handle esa_mode properly
@@ -786,6 +794,7 @@ def populate_geometric_factors(
786794
dataset[var].values = gf_dataset[col].values
787795

788796
# Update delta_minus and delta_plus based on ESA mode
797+
# converting eV to keV
789798
if esa_mode == 0: # HiRes
790799
dataset["energy_delta_minus"].values = energy_delta_hires_values
791800
dataset["energy_delta_plus"].values = energy_delta_hires_values

imap_processing/tests/lo/test_lo_l2.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1770,6 +1770,9 @@ def test_populate_geometric_factors(
17701770
assert result["geometric_factor_stat_uncert"].values[i] == (
17711771
1.5e-5 * (i + 1)
17721772
)
1773+
# Ensure that energy_deltas are in units of keV
1774+
assert np.all(result["energy_delta_plus"].values < 1)
1775+
assert np.all(result["energy_delta_minus"].values < 1)
17731776

17741777
def test_populate_geometric_factors_no_gf_species(self):
17751778
"""Test population for species without geometric factors."""

0 commit comments

Comments
 (0)