USACE / USACE/cwms-data-api

Add Options for resolving duplicate date-times when POSTing timeseries data

Open
#1,915 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Java
Stars
28
Forks
25
Avg merge
4d 22h
Merged PRs (30d)
20

Description

Is your feature request related to a problem? Please describe.
There is currently no way to resolve duplicate date-times in timeseries data. Current behavior is to error out with 400 error code.
See https://github.com/USACE/cwms-data-api/issues/1783

Describe the solution you'd like
Add options to specify first, last, average resolution if there are duplicate date-times present in timeseries data. Default would be to error.

Additional context

https://github.com/USACE/cwms-data-api/issues/1783

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the POST timeseries endpoint and trace where duplicate date-times currently produce the 400 response. Determine how first, last, and average resolution options should be exposed while retaining error as the default, then add coverage for each option and the default behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api, backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.