Skip to content

I-ALiRT - Add SC Status#2922

Merged
laspsandoval merged 1 commit into
IMAP-Science-Operations-Center:devfrom
laspsandoval:parse_spacecraft_packet
Apr 8, 2026
Merged

I-ALiRT - Add SC Status#2922
laspsandoval merged 1 commit into
IMAP-Science-Operations-Center:devfrom
laspsandoval:parse_spacecraft_packet

Conversation

@laspsandoval
Copy link
Copy Markdown
Contributor

This pull request introduces the new process_status function for generating L1 status data products from IALIRT L0 packet data, along with corresponding unit tests and updates to external test data configuration. The most significant changes are the addition of the core status processing logic, comprehensive tests for this functionality, and the inclusion of new test data files to support validation.

Status Processing Functionality:

  • Added a new process_status function in imap_processing/ialirt/l0/process_status.py that processes parsed L0 xarray datasets into a list of status dictionaries, handling time conversion, extracting instrument status fields, and formatting output for downstream use.

Testing Enhancements:

  • Introduced a new unit test module imap_processing/tests/ialirt/unit/test_process_status.py that verifies the correctness of the process_status function using multiple postlaunch packet files and asserts expected status values for each instrument.

Test Data Configuration:

  • Added five new postlaunch packet files to the test data configuration in imap_processing/tests/external_test_data_config.py to enable thorough testing of the status processing logic.

@laspsandoval laspsandoval added this to the April 2026 milestone Apr 6, 2026
@laspsandoval laspsandoval requested a review from lacoak21 April 6, 2026 15:17
@laspsandoval laspsandoval self-assigned this Apr 6, 2026
@laspsandoval laspsandoval added this to IMAP Apr 6, 2026
@laspsandoval laspsandoval linked an issue Apr 6, 2026 that may be closed by this pull request
@laspsandoval laspsandoval moved this to Canceled in IMAP Apr 6, 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.

Looks good!

logger = logging.getLogger(__name__)


def process_status(xarray_data: xr.Dataset) -> list[dict]:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Where is this going to be called?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It will be called in sds-data-manager (other PR: IMAP-Science-Operations-Center/sds-data-manager#1218)

@laspsandoval laspsandoval merged commit 8f74867 into IMAP-Science-Operations-Center:dev Apr 8, 2026
14 checks passed
@github-project-automation github-project-automation Bot moved this from Canceled to Done in IMAP Apr 8, 2026
@laspsandoval laspsandoval deleted the parse_spacecraft_packet branch April 8, 2026 14:54
sapols pushed a commit to sapols/imap_processing that referenced this pull request Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

I-ALiRT - Add Health and Safety : imap_processing

2 participants