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

Percentage calculations seem to be off #31

Closed
jasontempleman-eaton opened this issue May 16, 2024 · 3 comments
Closed

Percentage calculations seem to be off #31

jasontempleman-eaton opened this issue May 16, 2024 · 3 comments
Assignees

Comments

@jasontempleman-eaton
Copy link

Can someone please explain how the average calculations are computed?

Looking at the values in the readme for Cumulative Number of Accepted Prompts (50734) and Cumulative Number of Suggested Prompts (199097) I would expect the average would be 50734 / 199097 * 100 or 25.48% whereas the image shows 22.19%

I ran this locally again my Enterprise and the values are 20024 and 68238 so I would expect an average of 29.34% yet the value given is 20.54%

I would like to understand where these differences are coming from so that I can make sure I have the correct values.

@mbroton
Copy link

mbroton commented May 31, 2024

From what I see in src/components/MetricsViewer.vue there is calculation of acceptanceRateAverage done based on daily average value, instead of basing it on the total numbers. I would assume that's the issue, but I didn't dive too deep into the code, so I might be wrong.

@martedesco
Copy link
Collaborator

martedesco commented May 31, 2024

@mbroton - that's correct! I believe we should favor the total number of suggestions divided by the number of accepted prompts. I’m working on a fix for it.

@martedesco martedesco self-assigned this May 31, 2024
@martedesco
Copy link
Collaborator

Fixed in this PR. It will be available in the release v1.5 coming tomorrow. Thanks @jasontempleman-eaton for raising this.

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

3 participants