DevYoungHulk commented on issue #7873: URL: https://github.com/apache/incubator-devlake/issues/7873#issuecomment-2285388262
> > But PR1,PR2 are not really deployed until PR3 is completed, does that make sense to calculate? > > Why is that? You said the following in the issue description: > > > For example i have multiple PRs in one deployment: PR1 (commit1,commit2), PR2 (commit3,commit4), PR3 (commit5)" > > If I understand the statement correctly, it means: They were deployed together in one go. Yes, the 3 PRS are included in a single deployment. > From what I can tell, the Median Lead Time for Changes is calculated by: > > 1. Select deployments within the time range > 2. Select all PRs belonging to those deployments > 3. Calculate the Median from all Cycle Times of those PRs > > So, if the deployment gets selected in the first step, then yes, all 3 PRs would be included. So in my example, the lead time for change for this deployment is from P1 start time to go live time, P2, P3 time need not be double counted right? While calculating median we need to calculate as per deployment not PR. Am i right? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@devlake.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
