Skip to content

I-ALiRT - add since parameter#26

Merged
laspsandoval merged 2 commits into
mainfrom
2026_02_26_release
Mar 4, 2026
Merged

I-ALiRT - add since parameter#26
laspsandoval merged 2 commits into
mainfrom
2026_02_26_release

Conversation

@laspsandoval
Copy link
Copy Markdown
Contributor

This pull request adds support for a new --since parameter to the ialirt-archive-query CLI and underlying API, allowing users to query archive files from a specified date onward. It also enforces that --since cannot be combined with the existing date filters (--year, --month, --day). Documentation and tests have been updated to reflect and validate this new functionality.

New Feature: Archive Query by Date Range

  • Added a --since argument to the CLI (ialirt_data_access/cli.py) and the archive_query function (ialirt_data_access/io.py), enabling users to retrieve all files on or after a given date in YYYYMMDD format. [1] [2] [3]

Input Validation

  • Enforced that since cannot be combined with year, month, or day in the API, raising a ValueError if this rule is violated.

Documentation Updates

  • Updated CLI usage examples and documentation in README.md and ialirt_data_access/cli.py to describe the new --since parameter and its constraints. [1] [2]

Testing

  • Added new tests to verify correct behavior when using since, and to ensure errors are raised when since is combined with other date filters.

Internal Refactoring

  • Updated query parameter construction to include since only when provided, ensuring clean API requests.

These changes collectively enhance the flexibility of archive queries and improve user guidance and code robustness.

@laspsandoval laspsandoval self-assigned this Mar 2, 2026
@laspsandoval laspsandoval added this to the March 2026 milestone Mar 2, 2026
@laspsandoval laspsandoval requested a review from lacoak21 March 2, 2026 16:36
@laspsandoval laspsandoval merged commit d568ad2 into main Mar 4, 2026
15 checks passed
@laspsandoval laspsandoval deleted the 2026_02_26_release branch March 4, 2026 17:42
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