You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The new changes introduced in seem to have altered the behaviour of average acceptance rate calculations:
The new way has a much lower acceptance rate and its because of this change to the calculations:
total_lines_accepted/total_lines_suggested
instead of
total_acceptances_count/total_suggestions_count
Is this the correct behaviour now or a bug? I hope that the PRs title can reflect what was actually changed as end users it looks like the chart data suddenly changed on us without prior notice.
Previously in Oct this PR was introduced to use total_acceptances_count/total_suggestions_count to be inline with other plugins
[https://github.com/https://github.com//pull/1503]
But now its back to using total_lines_accepted/total_lines_suggested
Please let us know!
Thanks,
Eric
👍 Expected behavior
total_acceptances_count/total_suggestions_count
👎 Actual Behavior with Screenshots
total_lines_accepted/total_lines_suggested
👟 Reproduction steps
See screenshots above for the same date.
📃 Provide the context for the Bug.
The PR seems to have reverted the calcaculations for average acceptance rate back to lines of code. It would be nice if the PR mentioned if that is the intended change, so that us end users dont need to guess why the metrics suddenly changed on us.
👀 Have you spent some time to check if this bug has been raised before?
Plugin Name
Github Copilot
📜 Description
The new changes introduced in seem to have altered the behaviour of average acceptance rate calculations:
The new way has a much lower acceptance rate and its because of this change to the calculations:
total_lines_accepted/total_lines_suggested
instead of
total_acceptances_count/total_suggestions_count
Is this the correct behaviour now or a bug? I hope that the PRs title can reflect what was actually changed as end users it looks like the chart data suddenly changed on us without prior notice.
Previously in Oct this PR was introduced to use total_acceptances_count/total_suggestions_count to be inline with other plugins
[https://github.com/https://github.com//pull/1503]
But now its back to using total_lines_accepted/total_lines_suggested
Please let us know!
Thanks,
Eric
👍 Expected behavior
total_acceptances_count/total_suggestions_count
👎 Actual Behavior with Screenshots
total_lines_accepted/total_lines_suggested
👟 Reproduction steps
See screenshots above for the same date.
📃 Provide the context for the Bug.
The PR seems to have reverted the calcaculations for average acceptance rate back to lines of code. It would be nice if the PR mentioned if that is the intended change, so that us end users dont need to guess why the metrics suddenly changed on us.
👀 Have you spent some time to check if this bug has been raised before?
🏢 Have you read the Code of Conduct?
Are you willing to submit PR?
None
The text was updated successfully, but these errors were encountered: