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

Beginning of periods #3167

Merged
merged 10 commits into from
Oct 5, 2023
Merged

Beginning of periods #3167

merged 10 commits into from
Oct 5, 2023

Conversation

egor-ryashin
Copy link
Contributor

@egor-ryashin egor-ryashin commented Oct 3, 2023

See #3135

Benchmarks:

BenchmarkTimeSeries_hourly-10                             	      34	  30222012 ns/op	 1491715 B/op	   32548 allocs/op
BenchmarkTimeSeries_daily-10                              	      42	  26405661 ns/op	  171519 B/op	    3877 allocs/op
BenchmarkTimeSeries_weekly-10                             	      46	  26042627 ns/op	  123206 B/op	    2822 allocs/op
BenchmarkTimeSeries_weekly_first_day_of_week_Monday-10    	      44	  25638063 ns/op	  123120 B/op	    2822 allocs/op
BenchmarkTimeSeries_weekly_first_day_of_week_Sunday-10    	      45	  26494331 ns/op	  123498 B/op	    2824 allocs/op

@egor-ryashin egor-ryashin marked this pull request as ready for review October 4, 2023 15:15
@nishantmonu51 nishantmonu51 added the blocker A release blocker issue that should be resolved before a new release label Oct 4, 2023
@begelundmuller begelundmuller merged commit 02bafca into main Oct 5, 2023
5 checks passed
@begelundmuller begelundmuller deleted the first-day-of-week branch October 5, 2023 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker A release blocker issue that should be resolved before a new release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants