Skip to content

I-ALiRT - check coverage#2948

Merged
laspsandoval merged 1 commit intoIMAP-Science-Operations-Center:devfrom
laspsandoval:coverage_updates
Apr 9, 2026
Merged

I-ALiRT - check coverage#2948
laspsandoval merged 1 commit intoIMAP-Science-Operations-Center:devfrom
laspsandoval:coverage_updates

Conversation

@laspsandoval
Copy link
Copy Markdown
Contributor

@laspsandoval laspsandoval commented Apr 9, 2026

This pull request removes the parse_uksa_schedule_xlsx function and its associated tests from the codebase, as well as some unused imports. The changes help to simplify the generate_coverage.py module and its test suite by eliminating code that is no longer needed.

Code cleanup and simplification:

  • Removed the parse_uksa_schedule_xlsx function from generate_coverage.py, including all related logic for parsing UKSA (GHY-6) availability sheets.
  • Deleted all tests related to parse_uksa_schedule_xlsx from the test file test_generate_coverage.py, including both unit tests and integration tests that used this function.
  • Removed the import of parse_uksa_schedule_xlsx from the test file test_generate_coverage.py.
  • Cleaned up unused imports (Path and pandas as pd) from generate_coverage.py.

Data update:

  • Added "DSS-43" to the list of station names in generate_coverage.py.

Note: Originally this PR was intended to fix any differences between the predicted ephemeris kernel and the actual packet ingest timeframes. But after some analysis it was determined that most likely the recent predicted ephemeris is less accurate due to possible recent maneuvers.

│ Date │ Kernel era │ Start error │ End error │
│ 2026-03-12 │ March kernel │ +4 min │ -1 min │
│ 2026-04-04 │ od028 (Apr 3+) │ +18 min │ -15 min │

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! So clean 😍

@laspsandoval laspsandoval merged commit 7f33698 into IMAP-Science-Operations-Center:dev Apr 9, 2026
14 checks passed
@laspsandoval laspsandoval deleted the coverage_updates branch April 9, 2026 21:07
@github-project-automation github-project-automation bot moved this to Done in IMAP Apr 9, 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 - Compare predicted schedule plots for Kiel with actual connection times

2 participants