open-digger
open-digger copied to clipboard
[Metrics] Change Request Acceptance Ratio
Description Each change request can be in one of three states: open, merged (accepted), and closed without merge (declined). This metric measures the ratio of change requests merged (accepted) vs change requests closed without being merged.
Instance of Implementation
Resource CHAOSS: Evolution -> Code Development Activity
This issue has not been replied for 24 hours, please pay attention to this issue: @sunshinemingo @wengzhenjie
Hi, i'm trying to implement this mertic. I calculate the ratio in two ways,the first is ratio = pr closed without being merged/pr merged, the second is ratio = pr merged/pr closed(including pr closed without being merged and pr merged)。The result is shown below
But i think the result not represented the ratio of change requests merged (accepted) vs change requests closed without being merged well. Maybe I should use a histogram,just like the Instance of Implementation?Can anyone give me some advice? @xgdyp
I agree that the histogram is a better presentation here.
As for the value itself, I think the second way (merged/all closed) represents the meaning of this metric as "the ratio of accepted PRs among all the PRs"
closed by #1097