jirametrics 2.24 → 2.25pre7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ab33e299bd374f28754ecfbd750f6afd89861ecf64a323f28d4678d7a22959d
-  data.tar.gz: a0bf98e4d51b86eaa36aef8c6bfb69e8cdba958d3afeaf67da71a37ff6eb3ae5
+  metadata.gz: 98bcc110a47cd8dfe55eafb58b699d27604f2fb8d24a7ed357e6dd970998af8d
+  data.tar.gz: 2501f1064f33999d31ce42a1dcdf29fd023fb5b60fc06f01294d064d60210fc4
 SHA512:
-  metadata.gz: 7c017b109ce143403190db03124148def9be7a92fe20961f6f7c42f459482a4056018601f131d6ca6275889e7231e54df54befbccb4279de0073838d50280ff4
-  data.tar.gz: 07b783818d028d3d95fd1cd861f272fe624c46fe3d4b671f3e2e6fede8fcdc694e5118da888c7c12113838e37711ccd6935a54261d4edf5ceaac4e29c6a08a73
+  metadata.gz: 0761c74f71511217ed38496578d2863df6e5c1fe38d2932fd6fa3c1fed6dd626f5bcbc8bc015ce06aa91d098c55dbff5f1944daba386e39cdb9319f66d658cee
+  data.tar.gz: b3657601410caceb3b186994785609f55d67f80094c945cd34203f68ff8dcdd494ff3d51375498071cfe0f316bf69aef38eac75e3a0e3acfb1462c8b797dc5cd

data/lib/jirametrics/aging_work_bar_chart.rb

@@ -66,7 +66,7 @@ class AgingWorkBarChart < ChartBase
 
   def adjust_time_date_ranges_to_start_from_earliest_issue_start aging_issues
     earliest_start_time = aging_issues.collect do |issue|
-      issue.board.cycletime.started_stopped_times(issue).first
+      issue.started_stopped_times.first
     end.min
     return if earliest_start_time.nil? || earliest_start_time >= @time_range.begin
 
@@ -102,7 +102,7 @@ class AgingWorkBarChart < ChartBase
 
   def select_aging_issues issues:
     issues.select do |issue|
-      started_time, stopped_time = issue.board.cycletime.started_stopped_times(issue)
+      started_time, stopped_time = issue.started_stopped_times
       next false unless started_time && stopped_time.nil?
 
       age = (date_range.end - started_time.to_date).to_i + 1
@@ -128,7 +128,7 @@ class AgingWorkBarChart < ChartBase
 
   def collect_status_ranges issue:, now:
     ranges = []
-    issue_started_time = issue.board.cycletime.started_stopped_times(issue).first
+    issue_started_time = issue.started_stopped_times.first
     previous_start = nil
     previous_status = nil
     issue.status_changes.each do |change|

data/lib/jirametrics/aging_work_table.rb

@@ -50,15 +50,14 @@ class AgingWorkTable < ChartBase
 
   def expedited_but_not_started
     @issues.select do |issue|
-      started_time, stopped_time = issue.board.cycletime.started_stopped_times(issue)
+      started_time, stopped_time = issue.started_stopped_times
       started_time.nil? && stopped_time.nil? && issue.expedited?
     end.sort_by(&:created)
   end
 
   def select_aging_issues
     aging_issues = @issues.select do |issue|
-      cycletime = issue.board.cycletime
-      started, stopped = cycletime.started_stopped_times(issue)
+      started, stopped = issue.started_stopped_times
       next false if started.nil? || stopped
       next true if issue.blocked_on_date?(@today, end_time: time_range.end) || issue.expedited?
 
@@ -77,7 +76,7 @@ class AgingWorkTable < ChartBase
   end
 
   def blocked_text issue
-    started_time, _stopped_time = issue.board.cycletime.started_stopped_times(issue)
+    started_time, _stopped_time = issue.started_stopped_times
     return nil if started_time.nil?
 
     current = issue.blocked_stalled_changes(end_time: time_range.end)[-1]

data/lib/jirametrics/atlassian_document_format.rb

@@ -44,9 +44,9 @@ class AtlassianDocumentFormat
       when 'codeBlock'                 then ['<code>', '</code>']
       when 'date'
         [Time.at(node_attrs['timestamp'].to_i / 1000, in: @timezone_offset).to_date.to_s, nil]
-      when 'decisionItem'              then ['<li>', '</li>']
+      when 'decisionItem', 'listItem'  then ['<li>', '</li>']
       when 'decisionList'              then ['<div>Decisions<ul>', '</ul></div>']
-      when 'emoji'                     then [node_attrs['text'], nil]
+      when 'emoji', 'status'           then [node_attrs['text'], nil]
       when 'expand'                    then ["<div>#{node_attrs['title']}</div>", nil]
       when 'hardBreak'                 then ['<br />', nil]
       when 'heading'
@@ -55,7 +55,6 @@ class AtlassianDocumentFormat
       when 'inlineCard'
         url = node_attrs['url']
         ["[Inline card]: <a href='#{url}'>#{url}</a>", nil]
-      when 'listItem'                  then ['<li>', '</li>']
       when 'media'
         text = node_attrs['alt'] || node_attrs['id']
         ["Media: #{text}", nil]
@@ -65,7 +64,6 @@ class AtlassianDocumentFormat
       when 'panel'                     then ["<div>#{node_attrs['panelType'].upcase}</div>", nil]
       when 'paragraph'                 then ['<p>', '</p>']
       when 'rule'                      then ['<hr />', nil]
-      when 'status'                    then [node_attrs['text'], nil]
       when 'table'                     then ['<table>', '</table>']
       when 'tableCell'                 then ['<td>', '</td>']
       when 'tableHeader'               then ['<th>', '</th>']
@@ -87,38 +85,29 @@ class AtlassianDocumentFormat
     adf_node_render(node) do |n|
       node_attrs = n['attrs']
       case n['type']
-      when 'blockquote'                then ['', nil]
-      when 'bulletList'                then ['', nil]
-      when 'codeBlock'                 then ['', nil]
+      when 'blockquote', 'bulletList', 'codeBlock',
+           'mediaSingle', 'mediaGroup',
+           'orderedList', 'table', 'taskList'  then ['', nil]
       when 'date'
         [Time.at(node_attrs['timestamp'].to_i / 1000, in: @timezone_offset).to_date.to_s, nil]
       when 'decisionItem'              then ['- ', "\n"]
       when 'decisionList'              then ["Decisions:\n", nil]
-      when 'emoji'                     then [node_attrs['text'], nil]
+      when 'emoji', 'mention', 'status' then [node_attrs['text'], nil]
       when 'expand'                    then ["#{node_attrs['title']}\n", nil]
       when 'hardBreak'                 then ["\n", nil]
-      when 'heading'                   then ['', "\n"]
+      when 'heading', 'paragraph', 'tableRow' then ['', "\n"]
       when 'inlineCard'                then [node_attrs['url'], nil]
       when 'listItem'                  then ['- ', nil]
       when 'media'
         text = node_attrs['alt'] || node_attrs['id']
         ["Media: #{text}", nil]
-      when 'mediaSingle', 'mediaGroup' then ['', nil]
-      when 'mention'                   then [node_attrs['text'], nil]
-      when 'orderedList'               then ['', nil]
       when 'panel'                     then ["#{node_attrs['panelType'].upcase}\n", nil]
-      when 'paragraph'                 then ['', "\n"]
       when 'rule'                      then ["---\n", nil]
-      when 'status'                    then [node_attrs['text'], nil]
-      when 'table'                     then ['', nil]
-      when 'tableCell'                 then ['', "\t"]
-      when 'tableHeader'               then ['', "\t"]
-      when 'tableRow'                  then ['', "\n"]
+      when 'tableCell', 'tableHeader'  then ['', "\t"]
       when 'text'                      then [n['text'], nil]
       when 'taskItem'
         state = node_attrs['state'] == 'TODO' ? '☐' : '☑'
         ["#{state} ", "\n"]
-      when 'taskList'                  then ['', nil]
       else
         ["[Unparseable: #{n['type']}]\n", nil]
       end

data/lib/jirametrics/board_movement_calculator.rb

@@ -10,7 +10,7 @@ class BoardMovementCalculator
   end
 
   def moves_backwards? issue
-    started, stopped = issue.board.cycletime.started_stopped_times(issue)
+    started, stopped = issue.started_stopped_times
     return false unless started
 
     previous_column = nil
@@ -70,7 +70,7 @@ class BoardMovementCalculator
     @issues.filter_map do |issue|
       this_column_start = issue.first_time_in_or_right_of_column(this_column.name)&.time
       next_column_start = next_column.nil? ? nil : issue.first_time_in_or_right_of_column(next_column.name)&.time
-      issue_start, issue_done = issue.board.cycletime.started_stopped_times(issue)
+      issue_start, issue_done = issue.started_stopped_times
 
       # Skip if we can't tell when it started.
       next if issue_start.nil?

data/lib/jirametrics/chart_base.rb

@@ -86,6 +86,12 @@ class ChartBase
     "#{hours} hour#{'s' unless hours == 1}"
   end
 
+  def label_minutes minutes
+    return 'unknown' if minutes.nil?
+
+    "#{minutes} minute#{'s' unless minutes == 1}"
+  end
+
   def label_issues count
     "#{count} issue#{'s' unless count == 1}"
   end

data/lib/jirametrics/cycletime_histogram.rb

@@ -32,7 +32,7 @@ class CycletimeHistogram < TimeBasedHistogram
     stopped_issues = completed_issues_in_range include_unstarted: true
 
     # For the histogram, we only want to consider items that have both a start and a stop time.
-    stopped_issues.select { |issue| issue.board.cycletime.started_stopped_times(issue).first }
+    stopped_issues.select { |issue| issue.started_stopped_times.first }
   end
 
   def value_for_item issue

data/lib/jirametrics/cycletime_scatterplot.rb

@@ -40,14 +40,14 @@ class CycletimeScatterplot < TimeBasedScatterplot
   end
 
   def x_value item
-    item.board.cycletime.started_stopped_times(item).last
+    item.started_stopped_times.last
   end
 
   def y_value item
     item.board.cycletime.cycletime(item)
   end
 
-  def title_value item
+  def title_value item, rules: nil
     hint = @issue_hints&.fetch(item, nil)
     "#{item.key} : #{item.summary} (#{label_days(y_value(item))})#{" #{hint}" if hint}"
   end

data/lib/jirametrics/daily_view.rb

@@ -36,7 +36,7 @@ class DailyView < ChartBase
 
   def select_aging_issues
     aging_issues = issues.select do |issue|
-      started_at, stopped_at = issue.board.cycletime.started_stopped_times(issue)
+      started_at, stopped_at = issue.started_stopped_times
       started_at && !stopped_at
     end
 
@@ -73,7 +73,7 @@ class DailyView < ChartBase
 
   def make_blocked_stalled_lines issue
     today = date_range.end
-    started_date = issue.board.cycletime.started_stopped_times(issue).first&.to_date
+    started_date = issue.started_stopped_times.first&.to_date
     return [] unless started_date
 
     blocked_stalled = issue.blocked_stalled_by_date(
@@ -256,7 +256,8 @@ class DailyView < ChartBase
     description = issue.raw['fields']['description']
     return [] unless description
 
-    text = "<div class='foldable startFolded'>Description</div><div>#{atlassian_document_format.to_html(description)}</div>"
+    text = "<div class='foldable startFolded'>Description</div>" \
+           "<div>#{atlassian_document_format.to_html(description)}</div>"
     [[text]]
   end
 

data/lib/jirametrics/daily_wip_by_age_chart.rb

@@ -49,7 +49,7 @@ class DailyWipByAgeChart < DailyWipChart
   end
 
   def default_grouping_rules issue:, rules:
-    started, stopped = issue.board.cycletime.started_stopped_dates(issue)
+    started, stopped = issue.started_stopped_dates
 
     if stopped && started.nil? # We can't tell when it started
       @has_completed_but_not_started = true

data/lib/jirametrics/daily_wip_by_blocked_stalled_chart.rb

@@ -39,7 +39,7 @@ class DailyWipByBlockedStalledChart < DailyWipChart
   end
 
   def default_grouping_rules issue:, rules:
-    started, stopped = issue.board.cycletime.started_stopped_times(issue)
+    started, stopped = issue.started_stopped_times
     stopped_date = stopped&.to_date
     started_date = started&.to_date
 

data/lib/jirametrics/daily_wip_by_parent_chart.rb

@@ -26,11 +26,13 @@ class DailyWipByParentChart < DailyWipChart
   end
 
   def default_grouping_rules issue:, rules:
-    parent = issue.parent&.key
+    parent = issue.parent
     if parent
-      rules.label = parent
+      rules.label = parent.key
+      rules.label_hint = "#{parent.key} : #{parent.summary}"
     else
       rules.label = 'No parent'
+      rules.label_hint = 'No parent'
       rules.group_priority = 1000
       rules.color = '--body-background'
     end

data/lib/jirametrics/daily_wip_chart.rb

@@ -104,7 +104,8 @@ class DailyWipChart < ChartBase
         .select { |_issue, rules| rules.group == grouping_rule.group }
         .sort_by { |issue, _rules| issue.key_as_i }
         .collect { |issue, rules| "#{issue.key} : #{issue.summary.strip} #{rules.issue_hint}" }
-      title = ["#{display_label} (#{label_issues issue_strings.size})"] + issue_strings
+      title_label = grouping_rule.label_hint || display_label
+      title = ["#{title_label} (#{label_issues issue_strings.size})"] + issue_strings
 
       {
         x: date,
@@ -123,6 +124,7 @@ class DailyWipChart < ChartBase
     {
       type: 'bar',
       label: display_label,
+      label_hint: grouping_rule.label_hint,
       data: data,
       backgroundColor: background_color,
       borderColor: CssVariable['--wip-chart-border-color'],

data/lib/jirametrics/data_quality_report.rb

@@ -121,7 +121,7 @@ class DataQualityReport < ChartBase
 
   def initialize_entries
     @entries = @issues.filter_map do |issue|
-      started, stopped = issue.board.cycletime.started_stopped_times(issue)
+      started, stopped = issue.started_stopped_times
       next if stopped && stopped < time_range.begin
       next if started && started > time_range.end
 
@@ -274,7 +274,7 @@ class DataQualityReport < ChartBase
 
     started_subtasks = []
     entry.issue.subtasks.each do |subtask|
-      started_subtasks << subtask if subtask.board.cycletime.started_stopped_times(subtask).first
+      started_subtasks << subtask if subtask.started_stopped_times.first
     end
 
     return if started_subtasks.empty?
@@ -293,7 +293,7 @@ class DataQualityReport < ChartBase
       next unless settings['blocked_link_text'].include?(link.label)
 
       this_active = !entry.stopped
-      other_active = !link.other_issue.board.cycletime.started_stopped_times(link.other_issue).last
+      other_active = !link.other_issue.started_stopped_times.last
       next unless this_active && !other_active
 
       entry.report(
@@ -326,7 +326,7 @@ class DataQualityReport < ChartBase
     return unless entry.stopped
 
     subtask_labels = entry.issue.subtasks.filter_map do |subtask|
-      subtask_started, subtask_stopped = subtask.board.cycletime.started_stopped_times(subtask)
+      subtask_started, subtask_stopped = subtask.started_stopped_times
 
       if !subtask_started && !subtask_stopped
         "#{subtask_label subtask} (Not even started)"

data/lib/jirametrics/dependency_chart.rb

@@ -231,7 +231,7 @@ class DependencyChart < ChartBase
       elsif is_done
         line2 << 'Done'
       else
-        started_at = issue.board.cycletime.started_stopped_times(issue).first
+        started_at = issue.started_stopped_times.first
         if started_at.nil?
           line2 << 'Not started'
         else

data/lib/jirametrics/examples/standard_project.rb

@@ -67,19 +67,19 @@ class Exporter
           cycletime_histogram
 
           throughput_chart do
-            description_text '<h2>Number of items completed, grouped by issue type</h2>'
+            description_text <<~TEXT
+              <div>Throughput data is very useful for#{' '}
+                <a href="https://blog.mikebowler.ca/2024/06/02/probabilistic-forecasting/">probabilistic forecasting</a>,
+                to determine when we'll be done. Try it now with the
+                <a href="<%= throughput_forecaster_url %>" target="_blank" rel="noopener noreferrer">
+                Focused Objective throughput forecaster,</a> to see how long it would take to complete all of the
+                <%= @not_started_count %> items you currently have in your backlog.
+              </div>
+              <h2>Number of items completed, grouped by issue type</h2>'
+            TEXT
           end
-          throughput_chart do
-            header_text nil
+          throughput_by_completed_resolution_chart do
             description_text '<h2>Number of items completed, grouped by completion status and resolution</h2>'
-            grouping_rules do |issue, rules|
-              status, resolution = issue.status_resolution_at_done
-              if resolution
-                rules.label = "#{status.name}:#{resolution}"
-              else
-                rules.label = status.name
-              end
-            end
           end
 
           aging_work_in_progress_chart
@@ -91,6 +91,7 @@ class Exporter
           flow_efficiency_scatterplot if show_experimental_charts
           sprint_burndown
           estimate_accuracy_chart
+          expedited_chart
           dependency_chart
         end
       end

data/lib/jirametrics/expedited_chart.rb

@@ -50,7 +50,7 @@ class ExpeditedChart < ChartBase
     end
 
     if data_sets.empty?
-      '<h1 class="foldable">Expedited work</h1><p>There is no expedited work in this time period.</p>'
+      '<h1 class="foldable">Expedited work</h1><div>There is no expedited work in this time period.</div>'
     else
       wrap_and_render(binding, __FILE__)
     end

data/lib/jirametrics/flow_efficiency_scatterplot.rb

@@ -62,7 +62,9 @@ class FlowEfficiencyScatterplot < ChartBase
       create_dataset(issues: issues, label: rules.label, color: rules.color)
     end
 
-    return "<h1 class='foldable'>#{@header_text}</h1>No data matched the selected criteria. Nothing to show." if data_sets.empty?
+    if data_sets.empty?
+      return "<h1 class='foldable'>#{@header_text}</h1>No data matched the selected criteria. Nothing to show."
+    end
 
     wrap_and_render(binding, __FILE__)
   end

data/lib/jirametrics/github_gateway.rb

@@ -96,7 +96,7 @@ class GithubGateway
     # This extra check seems to only matter on Windows. On the mac, auth failures don't pass status.success?
     if stderr.include?('SAML enforcement')
       raise "GitHub CLI is not authorized to access #{@repo}. " \
-            "Run: gh auth refresh -h github.com -s read:org"
+            'Run: gh auth refresh -h github.com -s read:org'
     end
 
     raise "GitHub CLI command failed for #{@repo}: #{stderr}" unless status.success?

data/lib/jirametrics/grouping_rules.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 class GroupingRules < Rules
-  attr_accessor :label, :issue_hint
+  attr_accessor :label, :issue_hint, :label_hint
   attr_reader :color
 
   def eql? other

data/lib/jirametrics/html/daily_wip_chart.erb

@@ -3,6 +3,11 @@
   <canvas id="<%= chart_id %>" width="<%= canvas_width %>" height="<%= canvas_height %>"></canvas>
 </div>
 <script>
+if (!Chart.Tooltip.positioners.legendItem) {
+  Chart.Tooltip.positioners.legendItem = function(items) {
+    return this.chart._legendHoverPosition || Chart.Tooltip.positioners.average.call(this, items);
+  };
+}
 new Chart(document.getElementById('<%= chart_id %>').getContext('2d'),
 {
   type: 'bar',
@@ -43,9 +48,16 @@ new Chart(document.getElementById('<%= chart_id %>').getContext('2d'),
     },
     plugins: {
       tooltip: {
+        position: 'legendItem',
         callbacks: {
+          title: function(contexts) {
+            if (contexts[0]?.chart._legendHoverIndex != null) return '';
+          },
           label: function(context) {
-            return context.dataset.data[context.dataIndex].title
+            if (context.chart._legendHoverIndex != null) {
+              return context.dataset.label_hint || '';
+            }
+            return context.dataset.data[context.dataIndex].title;
           }
         }
       },
@@ -56,6 +68,26 @@ new Chart(document.getElementById('<%= chart_id %>').getContext('2d'),
         }
       },
       legend: {
+        onHover: function(event, legendItem, legend) {
+          const chart = legend.chart;
+          const dataset = chart.data.datasets[legendItem.datasetIndex];
+          if (!dataset?.label_hint) return;
+          chart._legendHoverIndex = legendItem.datasetIndex;
+          chart._legendHoverPosition = { x: event.x, y: event.y };
+          const firstNonZero = dataset.data.findIndex(d => d?.y !== 0);
+          if (firstNonZero === -1) return;
+          chart.tooltip.setActiveElements(
+            [{ datasetIndex: legendItem.datasetIndex, index: firstNonZero }],
+            { x: event.x, y: event.y }
+          );
+          chart.update();
+        },
+        onLeave: function(event, legendItem, legend) {
+          legend.chart._legendHoverIndex = null;
+          legend.chart._legendHoverPosition = null;
+          legend.chart.tooltip.setActiveElements([], { x: 0, y: 0 });
+          legend.chart.update();
+        },
         labels: {
           filter: function(item, chart) {
             // Logic to remove a particular legend item goes here

data/lib/jirametrics/html/index.css

@@ -87,6 +87,9 @@
 body {
   background-color: var(--body-background);
   color: var(--default-text-color);
+  font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol";
+  font-size: 14px;
+  line-height: 1.5;
 }
 
 dl, dd, dt {

data/lib/jirametrics/html/throughput_chart.erb

@@ -3,6 +3,11 @@
   <canvas id="<%= chart_id %>" width="<%= canvas_width %>" height="<%= canvas_height %>"></canvas>
 </div>
 <script>
+if (!Chart.Tooltip.positioners.legendItem) {
+  Chart.Tooltip.positioners.legendItem = function(items) {
+    return this.chart._legendHoverPosition || Chart.Tooltip.positioners.average.call(this, items);
+  };
+}
 new Chart(document.getElementById('<%= chart_id %>').getContext('2d'), {
   type: 'scatter',
   data: {
@@ -40,9 +45,16 @@ new Chart(document.getElementById('<%= chart_id %>').getContext('2d'), {
     },
     plugins: {
       tooltip: {
+        position: 'legendItem',
         callbacks: {
+          title: function(contexts) {
+            if (contexts[0]?.chart._legendHoverIndex != null) return '';
+          },
           label: function(context) {
-            return context.dataset.data[context.dataIndex].title
+            if (context.chart._legendHoverIndex != null) {
+              return context.dataset.label_hint || '';
+            }
+            return context.dataset.data[context.dataIndex].title;
           }
         }
       },
@@ -51,6 +63,28 @@ new Chart(document.getElementById('<%= chart_id %>').getContext('2d'), {
           <%= working_days_annotation %>
           <%= date_annotation %>
         }
+      },
+      legend: {
+        onHover: function(event, legendItem, legend) {
+          const chart = legend.chart;
+          const dataset = chart.data.datasets[legendItem.datasetIndex];
+          if (!dataset?.label_hint) return;
+          chart._legendHoverIndex = legendItem.datasetIndex;
+          chart._legendHoverPosition = { x: event.x, y: event.y };
+          const firstNonZero = dataset.data.findIndex(d => d?.y !== 0);
+          if (firstNonZero === -1) return;
+          chart.tooltip.setActiveElements(
+            [{ datasetIndex: legendItem.datasetIndex, index: firstNonZero }],
+            { x: event.x, y: event.y }
+          );
+          chart.update();
+        },
+        onLeave: function(event, legendItem, legend) {
+          legend.chart._legendHoverIndex = null;
+          legend.chart._legendHoverPosition = null;
+          legend.chart.tooltip.setActiveElements([], { x: 0, y: 0 });
+          legend.chart.update();
+        }
       }
     }
   }

data/lib/jirametrics/html/time_based_histogram.erb

@@ -1,3 +1,4 @@
+<div>
 <%= seam_start %>
 <div class="chart">
   <canvas id="<%= chart_id %>" width="<%= canvas_width %>" height="<%= canvas_height %>"></canvas>
@@ -119,3 +120,4 @@ if show_stats
 end
 %>
 <%= seam_end 'stats_table' %>
+</div>

data/lib/jirametrics/html_report_config.rb

@@ -20,21 +20,6 @@ class HtmlReportConfig < HtmlGenerator
     module_eval lines.join("\n"), __FILE__, __LINE__
   end
 
-  define_chart name: 'aging_work_bar_chart', classname: 'AgingWorkBarChart'
-  define_chart name: 'aging_work_table', classname: 'AgingWorkTable'
-  define_chart name: 'cycletime_scatterplot', classname: 'CycletimeScatterplot'
-  define_chart name: 'daily_wip_chart', classname: 'DailyWipChart'
-  define_chart name: 'daily_wip_by_age_chart', classname: 'DailyWipByAgeChart'
-  define_chart name: 'daily_wip_by_blocked_stalled_chart', classname: 'DailyWipByBlockedStalledChart'
-  define_chart name: 'daily_wip_by_parent_chart', classname: 'DailyWipByParentChart'
-  define_chart name: 'throughput_chart', classname: 'ThroughputChart'
-  define_chart name: 'expedited_chart', classname: 'ExpeditedChart'
-  define_chart name: 'cycletime_histogram', classname: 'CycletimeHistogram'
-  define_chart name: 'estimate_accuracy_chart', classname: 'EstimateAccuracyChart'
-  define_chart name: 'hierarchy_table', classname: 'HierarchyTable'
-  define_chart name: 'flow_efficiency_scatterplot', classname: 'FlowEfficiencyScatterplot'
-  define_chart name: 'daily_view', classname: 'DailyView'
-
   define_chart name: 'daily_wip_by_type', classname: 'DailyWipChart',
     deprecated_warning: 'This is the same as daily_wip_chart. Please use that one', deprecated_date: '2024-05-23'
   define_chart name: 'story_point_accuracy_chart', classname: 'EstimateAccuracyChart',
@@ -48,6 +33,25 @@ class HtmlReportConfig < HtmlGenerator
     @charts = [] # Where we store all the charts we executed so we can assert against them.
   end
 
+  def method_missing name, &block
+    class_name = name.to_s.split('_').map(&:capitalize).join
+    klass = Object.const_get(class_name)
+    raise NameError unless klass < ChartBase
+
+    block ||= ->(_) {}
+    execute_chart klass.new(block)
+  rescue NameError
+    super
+  end
+
+  def respond_to_missing? name, include_private = false
+    class_name = name.to_s.split('_').map(&:capitalize).join
+    klass = Object.const_get(class_name)
+    klass < ChartBase
+  rescue NameError
+    super
+  end
+
   def cycletime label = nil, &block
     @file_config.project_config.all_boards.each_value do |board|
       raise 'Multiple cycletimes not supported' if board.cycletime

data/lib/jirametrics/issue.rb

@@ -560,7 +560,7 @@ class Issue
   # return [number of active seconds, total seconds] that this issue had up to the end_time.
   # It does not include data before issue start or after issue end
   def flow_efficiency_numbers end_time:, settings: @board.project_config.settings
-    issue_start, issue_stop = @board.cycletime.started_stopped_times(self)
+    issue_start, issue_stop = started_stopped_times
     return [0.0, 0.0] if !issue_start || issue_start > end_time
 
     value_add_time = 0.0
@@ -753,12 +753,20 @@ class Issue
     end
   end
 
+  def started_stopped_times
+    board.cycletime.started_stopped_times(self)
+  end
+
+  def started_stopped_dates
+    board.cycletime.started_stopped_dates(self)
+  end
+
   def status_changes
     @changes.select { |change| change.status? }
   end
 
   def status_resolution_at_done
-    done_time = board.cycletime.started_stopped_times(self).last
+    done_time = started_stopped_times.last
     return [nil, nil] if done_time.nil?
 
     status_change = nil
@@ -812,6 +820,14 @@ class Issue
       created = parse_time(history['created'])
 
       history['items']&.each do |item|
+        if item['field'] == 'status' && item['to'].nil?
+          board.project_config.file_system.log(
+            "Issue #{key} has a status change without a 'to' id " \
+            "(from #{item['fromString'].inspect} to #{item['toString'].inspect}). Using id 0."
+          )
+          item = item.merge('to' => '0')
+        end
+
         @changes << ChangeItem.new(raw: item, time: created, author_raw: history['author'])
       end
     end

data/lib/jirametrics/issue_printer.rb

@@ -21,7 +21,7 @@ class IssuePrinter
     history = [] # time, type, detail
 
     if issue.board.cycletime
-      started_at, stopped_at = issue.board.cycletime.started_stopped_times(issue)
+      started_at, stopped_at = issue.started_stopped_times
       history << [started_at, nil, 'vvvv Started here vvvv', true] if started_at
       history << [stopped_at, nil, '^^^^ Finished here ^^^^', true] if stopped_at
     else

data/lib/jirametrics/project_config.rb

@@ -623,7 +623,7 @@ class ProjectConfig
       cutoff_time = block.call(issue)
       next if cutoff_time.nil?
 
-      original_start_time = issue.board.cycletime.started_stopped_times(issue).first
+      original_start_time = issue.started_stopped_times.first
       next if original_start_time.nil?
 
       issue.discard_changes_before cutoff_time

data/lib/jirametrics/pull_request_cycle_time_histogram.rb

@@ -0,0 +1,77 @@
+# frozen_string_literal: true
+
+require 'jirametrics/groupable_issue_chart'
+
+class PullRequestCycleTimeHistogram < TimeBasedHistogram
+  def initialize block
+    super()
+
+    @cycletime_unit = :days
+    @x_axis_title = 'Cycle time in days'
+
+    header_text 'PR Histogram'
+    description_text <<-HTML
+      <div class="p">
+        This cycletime Histogram shows how many pull requests completed in a certain timeframe. This can be
+        useful for determining how many different types of work are flowing through, based on the
+        lengths of time they take.
+      </div>
+    HTML
+
+    init_configuration_block(block) do
+      grouping_rules do |pull_request, _rule|
+        rules.label = pull_request.repo
+      end
+    end
+  end
+
+  def cycletime_unit unit
+    unless %i[minutes hours days].include?(unit)
+      raise ArgumentError, "cycletime_unit must be :minutes, :hours, or :days, got #{unit.inspect}"
+    end
+
+    @cycletime_unit = unit
+    @x_axis_title = "Cycle time in #{unit}"
+  end
+
+  def all_items
+    result = []
+    issues.each do |issue|
+      next unless issue.github_prs
+
+      issue.github_prs.each do |pr|
+        next unless pr.closed_at
+
+        result << pr
+      end
+    end
+    result.uniq
+  end
+
+  def value_for_item item
+    divisor = { minutes: 60.0, hours: 3600.0, days: 86_400.0 }[@cycletime_unit]
+    ((item.closed_at - item.opened_at) / divisor).ceil
+  end
+
+  def label_cycletime value
+    case @cycletime_unit
+    when :minutes then label_minutes(value)
+    when :hours then label_hours(value)
+    when :days then label_days(value)
+    end
+  end
+
+  def title_for_item count:, value:
+    "#{count} PR#{'s' unless count == 1} closed in #{label_cycletime value}"
+  end
+
+  def sort_items items
+    items.sort_by(&:opened_at)
+  end
+
+  def label_for_item item, hint:
+    label = "#{item.number} #{item.title}"
+    label << hint if hint
+    label
+  end
+end

data/lib/jirametrics/pull_request_cycle_time_scatterplot.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+require 'jirametrics/groupable_issue_chart'
+
+class PullRequestCycleTimeScatterplot < TimeBasedScatterplot
+  def initialize block
+    super()
+
+    @cycletime_unit = :days
+    @y_axis_title = 'Cycle time in days'
+
+    header_text 'Pull Request (PR) Scatterplot'
+    description_text <<-HTML
+      <div class="p">
+        This graph shows the cycle time for all closed pull requests (time from opened to closed).
+      </div>
+      #{describe_non_working_days}
+    HTML
+
+    init_configuration_block(block) do
+      grouping_rules do |pull_request, _rule|
+        rules.label = pull_request.repo
+      end
+    end
+  end
+
+  def cycletime_unit unit
+    unless %i[minutes hours days].include?(unit)
+      raise ArgumentError, "cycletime_unit must be :minutes, :hours, or :days, got #{unit.inspect}"
+    end
+
+    @cycletime_unit = unit
+    @y_axis_title = "Cycle time in #{unit}"
+  end
+
+  def all_items
+    result = []
+    issues.each do |issue|
+      issue.github_prs&.each do |pr|
+        result << pr if pr.closed_at
+      end
+    end
+    result
+  end
+
+  def x_value pull_request
+    pull_request.closed_at
+  end
+
+  def y_value pull_request
+    divisor = { minutes: 60, hours: 3600, days: 86_400 }[@cycletime_unit]
+    ((pull_request.closed_at - pull_request.opened_at) / divisor).round
+  end
+
+  def label_cycletime value
+    case @cycletime_unit
+    when :minutes then label_minutes(value)
+    when :hours then label_hours(value)
+    when :days then label_days(value)
+    end
+  end
+
+  def title_value pull_request, rules: nil
+    age_label = label_cycletime y_value(pull_request)
+    "#{pull_request.title} | #{rules.label} | Age:#{age_label}#{lines_changed_text(pull_request)}"
+  end
+
+  def lines_changed_text pull_request
+    return '' unless pull_request.changed_files
+
+    additions = pull_request.additions || 0
+    deletions = pull_request.deletions || 0
+    text = +' | Lines changed: ['
+    text << "+#{to_human_readable additions}" unless additions.zero?
+    text << ' ' if additions != 0 && deletions != 0
+    text << "-#{to_human_readable deletions}" unless deletions.zero?
+    text << "], Files changed: #{to_human_readable pull_request.changed_files}"
+    text
+  end
+
+end

data/lib/jirametrics/sprint_burndown.rb

@@ -134,7 +134,7 @@ class SprintBurndown < ChartBase
 
     estimate_display_name = current_board.estimation_configuration.display_name
 
-    issue_completed_time = issue.board.cycletime.started_stopped_times(issue).last
+    issue_completed_time = issue.started_stopped_times.last
     completed_has_been_tracked = false
 
     issue.changes.each do |change|

data/lib/jirametrics/status.rb

@@ -36,7 +36,7 @@ class Status
   end
 
   def self.from_raw raw
-    raise "raw cannot be nil" if raw.nil?
+    raise 'raw cannot be nil' if raw.nil?
 
     category_config = raw['statusCategory']
     raise "statusCategory can't be nil in #{category_config.inspect}" if category_config.nil?

data/lib/jirametrics/throughput_by_completed_resolution_chart.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require 'jirametrics/throughput_chart'
+
+class ThroughputByCompletedResolutionChart < ThroughputChart
+  def initialize block
+    super
+    header_text 'Throughput, grouped by completion status and resolution'
+    description_text nil
+  end
+
+  def default_grouping_rules issue, rules
+    status, resolution = issue.status_resolution_at_done
+    if resolution
+      rules.label = "#{status.name}:#{resolution}"
+      rules.label_hint = "Status: #{status.name.inspect}:#{status.id}, resolution: #{resolution.inspect}"
+    else
+      rules.label = status.name
+      rules.label_hint = "Status: #{status.name.inspect}:#{status.id}"
+    end
+  end
+end

data/lib/jirametrics/throughput_chart.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'cgi'
+
 class ThroughputChart < ChartBase
   include GroupableIssueChart
 
@@ -10,45 +12,54 @@ class ThroughputChart < ChartBase
 
     header_text 'Throughput Chart'
     description_text <<-TEXT
-      <div class="p">
-        This chart shows how many items we completed per week
+      <div>Throughput data is very useful for#{' '}
+        <a href="https://blog.mikebowler.ca/2024/06/02/probabilistic-forecasting/">probabilistic forecasting</a>,
+        to determine when we'll be done. Try it now with the
+        <a href="<%= throughput_forecaster_url %>" target="_blank" rel="noopener noreferrer">
+        Focused Objective throughput forecaster,</a> to see how long it would take to complete all of the
+        <%= @not_started_count %> items you currently have in your backlog.
       </div>
       #{describe_non_working_days}
     TEXT
     @x_axis_title = nil
     @y_axis_title = 'Count of items'
 
-
     init_configuration_block(block) do
-      grouping_rules do |issue, rule|
-        rule.label = issue.type
-        rule.color = color_for type: issue.type
-      end
+      grouping_rules { |issue, rule| default_grouping_rules(issue, rule) }
     end
   end
 
   def run
+    # This is saved as an instance variable so that it's accessible later when rendering the description text
+    @not_started_count = issues.count { |issue| issue.started_stopped_times.first.nil? }
+
     completed_issues = completed_issues_in_range include_unstarted: true
     rules_to_issues = group_issues completed_issues
     data_sets = []
-    if rules_to_issues.size > 1
-      data_sets << weekly_throughput_dataset(
-        completed_issues: completed_issues,
-        label: 'Totals',
-        color: CssVariable['--throughput_chart_total_line_color'],
-        dashed: true
-      )
-    end
+    total_data_set = weekly_throughput_dataset(
+      completed_issues: completed_issues,
+      label: 'Totals',
+      color: CssVariable['--throughput_chart_total_line_color'],
+      dashed: true
+    )
+    @throughput_samples = total_data_set[:data].collect { |d| d[:y] }
+    data_sets << total_data_set if rules_to_issues.size > 1
 
     rules_to_issues.each_key do |rules|
       data_sets << weekly_throughput_dataset(
-        completed_issues: rules_to_issues[rules], label: rules.label, color: rules.color
+        completed_issues: rules_to_issues[rules], label: rules.label, color: rules.color,
+        label_hint: rules.label_hint
       )
     end
 
     wrap_and_render(binding, __FILE__)
   end
 
+  def default_grouping_rules issue, rule
+    rule.label = issue.type
+    rule.color = color_for type: issue.type
+  end
+
   def calculate_time_periods
     first_day = @date_range.begin
     first_day = case first_day.wday
@@ -68,10 +79,13 @@ class ThroughputChart < ChartBase
     end
   end
 
-  def weekly_throughput_dataset completed_issues:, label:, color:, dashed: false
+  def weekly_throughput_dataset completed_issues:, label:, color:, dashed: false, label_hint: nil
     result = {
       label: label,
-      data: throughput_dataset(periods: calculate_time_periods, completed_issues: completed_issues),
+      label_hint: label_hint,
+      data: throughput_dataset(
+        periods: calculate_time_periods, completed_issues: completed_issues, label_hint: label_hint
+      ),
       fill: false,
       showLine: true,
       borderColor: color,
@@ -82,19 +96,33 @@ class ThroughputChart < ChartBase
     result
   end
 
-  def throughput_dataset periods:, completed_issues:
+  def throughput_forecaster_url
+    params = {
+      throughputMode: 'data',
+      samplesText: @throughput_samples.join(','),
+      storyLow: @not_started_count,
+      storyHigh: @not_started_count
+    }
+
+    query = params.map { |k, v| "#{k}=#{CGI.escape(v.to_s)}" }.join('&')
+    "https://focusedobjective.com/throughput?#{query}"
+  end
+
+  def throughput_dataset periods:, completed_issues:, label_hint: nil
     periods.collect do |period|
       closed_issues = completed_issues.filter_map do |issue|
-        stop_date = issue.board.cycletime.started_stopped_dates(issue).last
+        stop_date = issue.started_stopped_dates.last
         [stop_date, issue] if stop_date && period.include?(stop_date)
       end
 
       date_label = "on #{period.end}"
       date_label = "between #{period.begin} and #{period.end}" unless period.begin == period.end
 
-      { y: closed_issues.size,
+      with_label_hint = label_hint ? " with #{label_hint}" : ''
+      {
+        y: closed_issues.size,
         x: "#{period.end}T23:59:59",
-        title: ["#{closed_issues.size} items completed #{date_label}"] +
+        title: ["#{closed_issues.size} items closed#{with_label_hint} #{date_label}"] +
           closed_issues.collect do |_stop_date, issue|
             hint = @issue_hints&.fetch(issue, nil)
             "#{issue.key} : #{issue.summary}#{" #{hint}" if hint}"

data/lib/jirametrics/time_based_scatterplot.rb

@@ -30,7 +30,7 @@ class TimeBasedScatterplot < ChartBase
       label = rules.label
       color = rules.color
       percent_line = calculate_percent_line items_by_type
-      data = items_by_type.filter_map { |item| data_for_item(item) }
+      data = items_by_type.filter_map { |item| data_for_item(item, rules: rules) }
       data_sets << {
         label: "#{label} (85% at #{label_days(percent_line)})",
         data: data,
@@ -79,7 +79,7 @@ class TimeBasedScatterplot < ChartBase
     }
   end
 
-  def data_for_item item
+  def data_for_item item, rules: nil
     y = y_value(item)
     return nil if y < 1 # These will get called out on the quality report
 
@@ -88,7 +88,7 @@ class TimeBasedScatterplot < ChartBase
     {
       y: y,
       x: chart_format(x_value(item)),
-      title: [title_value(item)]
+      title: [title_value(item, rules: rules)]
     }
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jirametrics
 version: !ruby/object:Gem::Version
-  version: '2.24'
+  version: 2.25pre7
 platform: ruby
 authors:
 - Mike Bowler
@@ -130,6 +130,8 @@ files:
 - lib/jirametrics/jira_gateway.rb
 - lib/jirametrics/project_config.rb
 - lib/jirametrics/pull_request.rb
+- lib/jirametrics/pull_request_cycle_time_histogram.rb
+- lib/jirametrics/pull_request_cycle_time_scatterplot.rb
 - lib/jirametrics/pull_request_review.rb
 - lib/jirametrics/raw_javascript.rb
 - lib/jirametrics/rules.rb
@@ -141,6 +143,7 @@ files:
 - lib/jirametrics/status.rb
 - lib/jirametrics/status_collection.rb
 - lib/jirametrics/stitcher.rb
+- lib/jirametrics/throughput_by_completed_resolution_chart.rb
 - lib/jirametrics/throughput_chart.rb
 - lib/jirametrics/time_based_histogram.rb
 - lib/jirametrics/time_based_scatterplot.rb