Skip to content

feat(ls): add --match and --expand for dataset discovery - #175

Open
djgormley wants to merge 2 commits into
CHIMEFRB:mainfrom
WVURAIL:feat/ls-match-expand
Open

feat(ls): add --match and --expand for dataset discovery#175
djgormley wants to merge 2 commits into
CHIMEFRB:mainfrom
WVURAIL:feat/ls-match-expand

Conversation

@djgormley

Copy link
Copy Markdown
Contributor

Finding a dataset currently means walking the hierarchy one name at a time. This adds two options to ls:

  • --match filters the larger datasets of a scope (or of every scope, when none is given) by comma-separated, case-insensitive terms that must all appear in the combined "scope dataset" text.
  • --expand opens each matched larger dataset one level and lists the children with their parent, so a hit that is a container (e.g. complex_gains holding timestamped days) resolves to usable rows.

Both compose the existing functions.list REST calls; no new endpoints. A scope or dataset the server does not answer for is reported in a failed list (stderr warning, or a failed key with --json) rather than shown as empty; an empty map with unanswered queries exits 1. --expand without a scope or --match is refused, since it would open every dataset in the archive.

Docs updated; live tests for the new paths plus offline tests pinning the unanswered-vs-empty accounting.

@djgormley
djgormley marked this pull request as draft August 24, 2026 23:52
@djgormley
djgormley force-pushed the feat/ls-match-expand branch from 500a910 to cee12e4 Compare August 25, 2026 02:54
@djgormley
djgormley requested a review from tjzegmott August 25, 2026 17:14
@djgormley
djgormley marked this pull request as ready for review August 25, 2026 17:14
@djgormley djgormley self-assigned this Aug 25, 2026
@djgormley djgormley added the FEATURE New Feature Request label Aug 25, 2026
@djgormley
djgormley marked this pull request as draft August 25, 2026 21:21
@djgormley
djgormley force-pushed the feat/ls-match-expand branch from a79051a to b4d4f75 Compare August 25, 2026 21:38
@CHIMEFRB CHIMEFRB deleted a comment from codecov-commenter Aug 25, 2026
@djgormley
djgormley marked this pull request as ready for review August 25, 2026 21:46
@djgormley
djgormley enabled auto-merge August 25, 2026 21:46
@djgormley
djgormley disabled auto-merge August 26, 2026 15:45
@CHIMEFRB CHIMEFRB deleted a comment from codecov-commenter Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FEATURE New Feature Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant