Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Allow partial days in the date window #100

Merged
merged 4 commits into from
Sep 24, 2021

Conversation

cosimon
Copy link
Contributor

@cosimon cosimon commented Sep 24, 2021

Description of change

Allow partial days in the date window

Manual QA steps

  • Ran the tap using a date_window_size of 0.5 and it queried for half a day at a time.

Risks

Rollback steps

  • revert this branch

@cosimon cosimon merged commit 5315cba into master Sep 24, 2021
@cosimon cosimon deleted the feature/allow-partial-days-in-date-window branch September 24, 2021 19:43
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.

2 participants