Skip to content

Commit 32be12f

Browse files
authored
Merge pull request #34 from github-copilot-resources/fix/acceptance-rate-calc
Refactor acceptance rate calculation in MetricsViewer.vue
2 parents c5ccd1d + fb4e70f commit 32be12f

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

src/components/MetricsViewer.vue

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -252,14 +252,12 @@ export default defineComponent({
252252
}
253253
]
254254
};
255-
256-
let sum = 0;
255+
257256
const acceptanceRates = data.map((m: Metrics) => {
258257
const rate = m.total_lines_suggested !== 0 ? (m.total_lines_accepted / m.total_lines_suggested) * 100 : 0;
259-
sum += rate;
260258
return rate;
261259
});
262-
acceptanceRateAverage.value = sum / data.length;
260+
263261
264262
acceptanceRateChartData.value = {
265263
labels: data.map((m: Metrics) => m.day),
@@ -274,6 +272,12 @@ export default defineComponent({
274272
}
275273
]
276274
};
275+
276+
if(cumulativeNumberSuggestions.value === 0){
277+
acceptanceRateAverage.value = 0;
278+
} else {
279+
acceptanceRateAverage.value = cumulativeNumberAcceptances.value / cumulativeNumberSuggestions.value * 100;
280+
}
277281
278282
totalActiveUsersChartData.value = {
279283
labels: data.map((m: Metrics) => m.day),

0 commit comments

Comments
 (0)