Skip to content

Fix CDA GUI data query workflow#1788

Merged
krowvin merged 24 commits into
developfrom
1721-cda-gui-updates
Jun 24, 2026
Merged

Fix CDA GUI data query workflow#1788
krowvin merged 24 commits into
developfrom
1721-cda-gui-updates

Conversation

@krowvin

@krowvin krowvin commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR got a bit large. I would be working through one issue and decide to fix another item i've been holding off on. See screenshots below for an exhaustive list of fixes / improvements.

Updates the CDA GUI data query workflow to make guided selection, time-series loading, and result rendering more reliable and usable.

Highlights:

  • Adds smarter guided location search across offices and preserves selected locations while search results change.
  • Uses interval-aware default lookback windows so coarse intervals request enough data.
  • Checks catalog extents before querying and offers suggested start dates when requested dates predate available data.
  • Chunks large time-series requests and shows per-series progress.
  • Virtualizes/optimizes table rendering for large result sets.
  • Moves failed/no-data status messages into the TimeSeries list and uses blue for loading state, reserving yellow for warnings.
  • Cleans up result controls and table styling.
  • Fixed issue with sub locations not rendering in the manual mode (reported by SWL)

Related Issue

Closes #1721

Validation

  • npm run lint
  • npm run format:check
  • git diff --check
  • In-app browser smoke checks on http://127.0.0.1:5173/cwms-data/data-query
  • Walked through and tested ever fix in localhost targeting prod CDA

Screenshots

  1. Improved Search in Guided Mode / Query mode select UI update / Moved office requirement to only "manual mode"
image
  1. Button to clear a selected location in guided mode
image
  1. If only a single option is presented in guided mode that will get preselected
image
  1. Ability to include missing TS in a request (toggles CDA flag) and improved UI for download / refresh buttons
image
  1. Added suggestion if a user tries to select a date before the start extents
image
  1. Added preselected kinds (option for all) in the guided search to narrow catalog. Helps given an office is not enforced to be selected in guided mode now
image
  1. Allow selecting more parameters if they match the other selected parts of the time series
image
  1. Move error messages inline with the timeseries selected
image Show missing TS was selected in the settings (item # 4) to render this
  1. Table was virtualized to improve load times and scrolling in large datasets
image
  1. Created date chunking to breakup dates by dates based on intervals and created better user feedback during this loadstate
image

Checklist

  • AI tools used

krowvin and others added 19 commits May 8, 2026 00:09
Co-authored-by: Codex <noreply@openai.com>
Signed-off-by: Charles Graham, SWT <charles.r.graham@usace.army.mil>
Signed-off-by: Charles Graham, SWT <charles.r.graham@usace.army.mil>
Signed-off-by: Charles Graham, SWT <charles.r.graham@usace.army.mil>
Signed-off-by: Charles Graham, SWT <charles.r.graham@usace.army.mil>
Signed-off-by: Charles Graham, SWT <charles.r.graham@usace.army.mil>
Signed-off-by: Charles Graham, SWT <charles.r.graham@usace.army.mil>
Signed-off-by: Charles Graham, SWT <charles.r.graham@usace.army.mil>
Signed-off-by: Charles Graham, SWT <charles.r.graham@usace.army.mil>
Signed-off-by: Charles Graham, SWT <charles.r.graham@usace.army.mil>
@krowvin krowvin linked an issue Jun 18, 2026 that may be closed by this pull request
@krowvin krowvin requested a review from MikeNeilson June 18, 2026 04:15

@MikeNeilson MikeNeilson left a comment

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.

Seems reasonable so far. Somewhat large number of changes, but a lot of that appears to be cleanup and reorganization.

Comment thread cda-gui/src/pages/data-query/components/TimeSeriesManager.jsx
@krowvin krowvin marked this pull request as ready for review June 19, 2026 22:24
@krowvin

krowvin commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator Author

Let me know if you have any concerns! Or if you're looking to do another review before approving or requesting changes @MikeNeilson

This does also resolve some feedback we got on improving the query tool.

i.e.
Easier to select gages and locations in the guided mode from any office if you don't know the office.

@krowvin krowvin merged commit 12a94f6 into develop Jun 24, 2026
8 checks passed
@krowvin krowvin deleted the 1721-cda-gui-updates branch June 24, 2026 00:52
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.

CDA GUI Updates

2 participants