Skip to content

I-ALiRT - added utc to packet query#27

Merged
laspsandoval merged 6 commits intoIMAP-Science-Operations-Center:mainfrom
laspsandoval:api_updates
Mar 4, 2026
Merged

I-ALiRT - added utc to packet query#27
laspsandoval merged 6 commits intoIMAP-Science-Operations-Center:mainfrom
laspsandoval:api_updates

Conversation

@laspsandoval
Copy link
Contributor

This pull request introduces support for two mutually exclusive query modes to the packet query functionality: traditional individual date/time parameters and a new ISO 8601 UTC time range mode. The CLI, core logic, and documentation are updated to reflect these enhancements, and new tests are added to validate the UTC range mode.

Packet query mode enhancements:

  • Added support for querying packets by UTC time range (time_utc_start and optional time_utc_end) in addition to the existing individual date/time parameters in ialirt_data_access/cli.py and ialirt_data_access/io.py. [1] [2]
  • Updated CLI argument parsing to make year and doy optional, and introduced time_utc_start and time_utc_end as new arguments for the UTC range mode. [1] [2]
  • Modified internal query parameter handling to support both modes and ensure only relevant parameters are passed to the backend. [1] [2] [3]

Documentation updates:

  • Expanded the README.md to describe both query modes, including usage examples for each.
  • Updated docstrings in ialirt_data_access/io.py to clarify parameter requirements and behavior for both modes.

Testing improvements:

  • Added new tests in tests/test_io.py to verify correct behavior for UTC range queries, including cases with only time_utc_start and with both start and end times.

@laspsandoval laspsandoval requested a review from lacoak21 March 2, 2026 19:14
@laspsandoval laspsandoval self-assigned this Mar 2, 2026
@laspsandoval laspsandoval added this to the March 2026 milestone Mar 2, 2026
@lacoak21
Copy link

lacoak21 commented Mar 4, 2026

Wow those links to the code in the description are great

@laspsandoval
Copy link
Contributor Author

pre-commit.ci autofix

@laspsandoval
Copy link
Contributor Author

pre-commit.ci autofix

@laspsandoval laspsandoval merged commit eaa9edb into IMAP-Science-Operations-Center:main Mar 4, 2026
13 checks passed
@laspsandoval laspsandoval deleted the api_updates branch March 4, 2026 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants