Skip to content

fix: allow_partial interval batching#3898

Merged
tobymao merged 1 commit intomainfrom
toby/partial_missing
Feb 25, 2025
Merged

fix: allow_partial interval batching#3898
tobymao merged 1 commit intomainfrom
toby/partial_missing

Conversation

@tobymao
Copy link
Copy Markdown
Contributor

@tobymao tobymao commented Feb 25, 2025

partial intervals were getting lumped into the previous interval expanding the range, this makes it so that the previous complete interval is isolated and the partial is new. for example

missing_interval('2023-01-01', '2023-01-02 12:00:00') should result in 2 intervals, [2023-01-01, 2023-01-02], [2023-01-02, 2023-01-02 12:00:00], but before this fix you only got one [2023-01-01, 2023-01-02 12:00:00]

closes #3859

@tobymao tobymao requested a review from izeigerman February 25, 2025 22:59
partial intervals were getting lumped into the previous interval
expanding the range, this makes it so that the previous complete
interval is isolated and the partial is new. for example

missing_interval('2023-01-01', '2023-01-02 12:00:00') should result in 2
intervals, [2023-01-01, 2023-01-02], [2023-01-02, 2023-01-02 12:00:00],
but before this fix you only got one [2023-01-01, 2023-01-02 12:00:00]

closes #3859
@tobymao tobymao force-pushed the toby/partial_missing branch from 7196d56 to 4ec555c Compare February 25, 2025 23:00
@tobymao tobymao merged commit 3e30179 into main Feb 25, 2025
@tobymao tobymao deleted the toby/partial_missing branch February 25, 2025 23:15
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.

Allow_partials option is updating last completed interval too

2 participants