ldmoose opened a new issue, #6944:
URL: https://github.com/apache/incubator-devlake/issues/6944

   ### Search before asking
   
   - [X] I had searched in the 
[issues](https://github.com/apache/incubator-devlake/issues?q=is%3Aissue) and 
found no similar issues.
   
   
   ### What happened
   
   Working:
   
![image](https://github.com/apache/incubator-devlake/assets/35307476/5e8f7bc2-9d66-4f2a-a5f3-67d1e30f22cb)
   
   Not Working:
   
![image](https://github.com/apache/incubator-devlake/assets/35307476/76f98323-1171-47ed-af58-f294a6683f90)
   
   
   ### What do you expect to happen
   
   I expect that when the time interval option in the board changes that the PR 
Cycle Time graph respects that change.
   
   ### How to reproduce
   
   1) Go to 
https://devlake.apache.org/livedemo/EngineeringLeads/EngineeringThroughputAndCycleTime
   2) Change Time Interval to 'Week'
   3) The Pr Cycle Time doesn't respect the change
   
   ### Anything else
   
   Problem is here: 
https://github.com/apache/incubator-devlake/blob/main/grafana/dashboards/EngineeringThroughputAndCycleTime.json#L1161
   
   Specifically, in the main select statement, the first column definition (and 
the subsequent group by) should have `INTERVAL 
-$interval(date(pr_issued_date))+1 DAY` as the second function parameter in the 
`DATE_ADD` function call.
   ```SQL
   WITH _prs AS (
     SELECT pr.id
       , pr.created_date AS pr_issued_date
       -- convert null to 0 if a PR has no cycle_time to make sure cycle_time 
equals the sum of the four metrics below
       , COALESCE(prm.pr_cycle_time/60,0) AS cycle_time
   FROM pull_requests pr
   LEFT JOIN project_pr_metrics prm ON pr.id = prm.id
   INNER JOIN project_mapping pm ON pr.base_repo_id = pm.row_id AND pm.table = 
'repos'
   WHERE $__timeFilter(pr.created_date)
   AND pr.created_date >= DATE_ADD(DATE_ADD($__timeFrom(), INTERVAL 
-DAY($__timeFrom())+1 DAY), INTERVAL +1 MONTH)
   AND pm.project_name in (${project:sqlstring}+'')
   GROUP BY pr.id
     , pr.created_date
     , COALESCE(prm.pr_cycle_time/60,0)
   )
   
   SELECT 
     DATE_ADD(date(pr_issued_date), INTERVAL 
-DAYOFMONTH(date(pr_issued_date))+1 DAY) AS time
     , AVG(cycle_time) AS 'PR Cycle Time(h)'
   FROM _prs
   GROUP BY DATE_ADD(date(pr_issued_date), INTERVAL 
-DAYOFMONTH(date(pr_issued_date))+1 DAY)
   ORDER BY time
   ```
   
   ### Version
   
   Latest: v0.19.0
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of 
Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
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