Skip to content

2374 lo l2 integrate cg correction for hf maps - prepare for lo integration#2381

Merged
tmplummer merged 10 commits intoIMAP-Science-Operations-Center:devfrom
tmplummer:2374-lo-l2---integrate-cg-correction-for-hf-maps
Nov 5, 2025
Merged

2374 lo l2 integrate cg correction for hf maps - prepare for lo integration#2381
tmplummer merged 10 commits intoIMAP-Science-Operations-Center:devfrom
tmplummer:2374-lo-l2---integrate-cg-correction-for-hf-maps

Conversation

@tmplummer
Copy link
Copy Markdown
Contributor

Change Summary

Overview

There are two main changes in this PR:

  • Make CG related functions in corrections.py act on xarray.Dataset objects rather than LoHiBasePsetSubclass objects. This need was driven by how the Lo L2 workflow is implemented (which I like much better than how I did it in Hi).
  • Move the function for calculating the pset ram_mask variable into the corrections.py module.

Other changes were cascaded from the above main changes:

  • Update Hi L2 processing workflow to only instantiate the HiPointingSet object just for use with the SkyMap projection method. (I will write up a ticket to extract functions similar to the Lo workflow)
  • Add more extensive tests for calculate_ram_mask function.

@tmplummer tmplummer self-assigned this Nov 4, 2025
@tmplummer tmplummer added Ins: Hi Related to the IMAP-Hi instrument Ins: Lo Related to the IMAP-Lo instrument labels Nov 4, 2025
@tmplummer tmplummer added this to IMAP Nov 4, 2025
@tmplummer tmplummer added the Mapper Tools Work related to common mapper tools label Nov 4, 2025
@tmplummer tmplummer force-pushed the 2374-lo-l2---integrate-cg-correction-for-hf-maps branch from f15677b to 16f9bff Compare November 4, 2025 21:34
Comment thread imap_processing/ena_maps/ena_maps.py Outdated
Comment thread imap_processing/ena_maps/utils/corrections.py
)

# Convert to Cartesian coordinates and store as DataArray
pset.data["look_direction"] = xr.DataArray(
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I really like the dropping of all these "data" and "values" :)

@tmplummer tmplummer force-pushed the 2374-lo-l2---integrate-cg-correction-for-hf-maps branch from 16f9bff to a9c0db1 Compare November 4, 2025 22:53
@tmplummer tmplummer merged commit 53a6cd0 into IMAP-Science-Operations-Center:dev Nov 5, 2025
14 checks passed
@github-project-automation github-project-automation bot moved this to Done in IMAP Nov 5, 2025
@tmplummer tmplummer deleted the 2374-lo-l2---integrate-cg-correction-for-hf-maps branch November 5, 2025 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ins: Hi Related to the IMAP-Hi instrument Ins: Lo Related to the IMAP-Lo instrument Mapper Tools Work related to common mapper tools

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants