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

[feature] Show percents rather than absolute values in charts. #85

Open
pmrv opened this issue Oct 2, 2016 · 2 comments
Open

[feature] Show percents rather than absolute values in charts. #85

pmrv opened this issue Oct 2, 2016 · 2 comments

Comments

@pmrv
Copy link

pmrv commented Oct 2, 2016

Especially for the "Days of the Week" chart it helps a lot more to see relative data than absolute numbers, because doing X on 70 Tuesdays might be a lot if only 80 Tuesdays passed or might be the opposite if instead 300 Tuesdays passed.
So I'd like to see a toggle to switch between absolute and relative display in the charts. Would you be interested in adding something like this? If so, I would look into it and add PR a couple weeks down the road.

@lordi
Copy link
Owner

lordi commented Oct 3, 2016

For the weekdays chart, I support the idea. There I would even prefer to show percentages without the ability to switch. For the other charts, it is quite unclear what percentage should mean (percentage of total ticks?, percentage of ticks in the visible time frame?, ...), so in order to avoid confusion I would keep the other charts as they are. What do you think?

I would appreciate a PR for percentage display in the weekdays chart.

@pmrv
Copy link
Author

pmrv commented Oct 5, 2016

For, say the weekly display, I would have used 7 as a base, for months 31/30 and so on. So you always get the average number of ticks per day, but it's not as straightforward as for the weekdays, that's true and since the numbers are a lot smaller there anyway, it's not that important actually.

But I will look into the weekdays first and maybe we can take it from there.

cipo7741 added a commit to cipo7741/tickmate that referenced this issue Jul 25, 2021
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

No branches or pull requests

2 participants