Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[BUG REPORT]: CWMS Timeseries Listings #210

Closed
ScottMassa opened this issue Nov 22, 2024 · 1 comment · Fixed by USACE/instrumentation-ui#246
Closed

[BUG REPORT]: CWMS Timeseries Listings #210

ScottMassa opened this issue Nov 22, 2024 · 1 comment · Fixed by USACE/instrumentation-ui#246
Assignees
Labels
bug Something isn't working

Comments

@ScottMassa
Copy link

Describe the bug

When viewing certain listings for timeseries, not all records are retrieved.

Steps To Reproduce

  1. Go to CWMS Timeseries
  2. Click on New CWMS Timesieres
  3. Select office "SWF"
  4. See projects only go through the letter "H"

Expected behavior

CWMS Timesiere lists all the projects

Actual behavior

The query has a page size of 5000, which is not enough to show all the listings. There are currently 10964, so a page size of say 15000 may be more appropriate.
https://cwms-data.usace.army.mil/cwms-data/catalog/TIMESERIES?office=SWF&page-size=5000](https://cwms-data.usace.army.mil/cwms-data/catalog/TIMESERIES?office=SWF&page-size=5000)

Screenshots

List stops at letter H
image

Desktop (please complete the following information):

  • OS: Windows 11
  • Browser Microsoft Edge
  • Version 131
@ScottMassa ScottMassa added the bug Something isn't working label Nov 22, 2024
@dennisgsmith
Copy link
Contributor

@KevinJJackson

Ref API Doc: https://cwms-data.usace.army.mil/cwms-data/swagger-ui.html

The catalog endpoint has a like query parameter for searching timesieres. I think the best way to resolve this would be to add that query param if there's text in the query string and debounce requests.

For scrolling the param list, there's a page query param that takes a hash of the previous request's next-page param.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants