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

[Bug] Calculation of average values in new statistics page is not correct #100

Closed
LordElamak opened this issue Oct 15, 2023 · 4 comments
Closed
Assignees
Labels
bug Something isn't working done

Comments

@LordElamak
Copy link

I've noticed that the average values in the statistics page are not correct.
For the average values of a week, the total value is always divided by 6 instead of 7.
I've also noticed this in September month statistics where the total values are divided by 29 isntead of 30.
Strangely the averages in October seems to be correct (total divided by 31).

In general, the calculation of the daily average values should be as follows:
average value = total value / (period end date - period start date +1)

Please can you check the calculation.

Furthermore I would expect that the average values of current periods (current week, month or year) will be calculated as "to date" values (divided by the number of days that passed from start of period till today). Currently these average values are calculated over the complete period, which results in low and meaningless values in my opinion, especially for the current year.

@LordElamak LordElamak added the bug Something isn't working label Oct 15, 2023
@Ayagikei
Copy link
Owner

Hello, thanks for your feedback!

We'll take a look asap.

@Ayagikei Ayagikei added the done label Oct 16, 2023
@Ayagikei
Copy link
Owner

We fixed the average calculation issue in v1.93.0-rc02.

But we haven't modified the "to date" values yet. This is a nice suggestion and we will continue to improve it in future versions.

@Ayagikei
Copy link
Owner

Furthermore I would expect that the average values of current periods (current week, month or year) will be calculated as "to date" values (divided by the number of days that passed from start of period till today). Currently these average values are calculated over the complete period, which results in low and meaningless values in my opinion, especially for the current year.

These changes will be released in v1.93.4, which we expect to release in the next week or two.

@Ayagikei Ayagikei added the to be released fixed but not released yet label Dec 25, 2023
@Ayagikei
Copy link
Owner

It should have been fixed in version 1.93.4.

@Ayagikei Ayagikei removed the to be released fixed but not released yet label Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working done
Projects
None yet
Development

No branches or pull requests

2 participants