jeroenheijmans / advent-of-code-charts

Inject charts in your private leaderboard page for Advent of Code
MIT License
120 stars 21 forks source link

Option to change cap on "Minutes taken per star" graph #98

Open Lexicality opened 10 months ago

Lexicality commented 10 months ago

First off thank you for this extension, it's very fun.

My problem is with the 4th graph, "Minutes taken per star". With timezones being what they are, people on my leaderboard tend not to start working on a day's challenge until at least 4 hours after the puzzle comes out, meaning that chart looks like this: image

This isn't very useful, other than to tell us that Steve wakes up earlier than everyone else.

It would be great if it was possible to change the cap, for example 24hrs would work much better for our use case.

Thanks

jeroenheijmans commented 10 months ago

Thanks for your feedback!

It actually used to be like that by default, but that made it super hard to do a precise and fun comparison with leaderboard competitors, because all stuff under 90 minutes was indistinguishable.

The option to configure this yourself might be interesting though, but I'm wary of what % of my users would want this. Not sure.

You spark another idea though: a toggle to have this graph show bars for only delta times between part 2 and 1 might be a good middle ground? That is: I live in Amsterdam time and have the same issue as you with this chart, but me and my colleagues do "compete" on the delta times a little bit.

I'll let your suggestion as well as the alternatives sit for a short bit to think on it some more. If others want to chime in that's welcome too!

Lexicality commented 10 months ago

Another thought I had was the ability to move were "zero" is, eg make it 8am UTC to bring everything back down, but that then means that if someone wakes up at 7am and does the day early then the graph could go negative which would be very strange.