Startrekzky commented on PR #8574:
URL: 
https://github.com/apache/incubator-devlake/pull/8574#issuecomment-3353065503

   Hi @Bhoopalan1999 , I checked the dashboard and it's overall good. 
   
   1. My biggest question is why there is a `cycle time` when `Issue lead time` 
already exists? They're the same to me.
   
   I checked the SQL and found you joined tables like 
`_tool_jira_issue_changelogs` and excluded specific issue types such as 'In 
Progress' and 'Developing'. This seems applicable only to your specific cases 
rather than a general approach. My suggestion is to remove this section if you 
want to get this merged to the main branch, which won't affect your ux, as you 
can still keep this section on your local dashboard.
   
   <img width="2836" height="1094" alt="image" 
src="https://github.com/user-attachments/assets/46503f50-8092-44de-8b10-780e3987c29f";
 />
   
   
   2. The number of overall issue delivery rate is not consistent with the rate 
by sprint
   I didn't dig into it. Please take a look.
   
   <img width="2868" height="1306" alt="image" 
src="https://github.com/user-attachments/assets/b710a9a9-eee8-4a77-8bf0-6104575824fb";
 />
   
   
   3. Please remove all status/type/... filters only applicable to your own use 
cases.
   Take 'delivery rate' as an example, I checked the SQL and found that you 
added the condition to exclude 'cancelled' status. This is not recommended if 
you want to share this with other users.
   
   <img width="2264" height="994" alt="image" 
src="https://github.com/user-attachments/assets/54cccbbd-b561-41e2-bb93-2fba01914879";
 />
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to