Merge pull request #33 from pfizer-opensource/unused-logic-removal

Remove data processor from the exported dist code

Author: ClaudiaGivan

src/graphs/UIControlsRenderer.js

@@ -96,7 +96,6 @@ export default class UIControlsRenderer extends Renderer {
   computeReportingRange(noOfDays) {
     const finalDate = this.data[this.data.length - 1][this.datePropertyName];
     let endDate = new Date(finalDate);
-    console.log(this.data[this.data.length - 1], finalDate, noOfDays);
     let startDate = addDaysToDate(finalDate, -Number(noOfDays));
     if (this.selectedTimeRange) {
       endDate = new Date(this.selectedTimeRange[1]);
@@ -155,8 +154,6 @@ export default class UIControlsRenderer extends Renderer {
    * and then redraws the x-axis based on the selected time range.
    */
   changeTimeInterval(isManualUpdate, chart) {
-    // console.log("this.timeInterval", this.timeInterval)
-
     if (isManualUpdate) {
       switch (this.timeInterval) {
         case 'weeks':
@@ -178,7 +175,6 @@ export default class UIControlsRenderer extends Renderer {
   }
 
   determineTheAppropriateAxisLabels() {
-    // console.log("this.reportingRangeDays", this.reportingRangeDays)
     if (this.reportingRangeDays <= 31) {
       return 'days';
     }

src/graphs/cfd/CFDRenderer.js

@@ -53,7 +53,6 @@ class CFDRenderer extends UIControlsRenderer {
     super(data);
     this.states = states;
     this.#statesColors = d3.scaleOrdinal().domain(this.states).range(this.#colorPalette);
-    console.table(this.data);
   }
 
   /**
@@ -568,21 +567,12 @@ class CFDRenderer extends UIControlsRenderer {
       const xPosition = coords[0];
       const yPosition = coords[1];
 
-      // Debug logs
-      // console.log("coords:", coords);
-      // console.log("xPosition:", xPosition, "yPosition:", yPosition);
-      // console.log("currentXScale domain:", this.currentXScale.domain());
-      // console.log("currentXScale range:", this.currentXScale.range());
-
-      // Ensure xPosition is within the chart's range
       if (xPosition < 0 || xPosition > this.width) {
-        console.log('xPosition out of bounds:', xPosition);
+        console.warn('Mouse position out of bounds:', xPosition);
         return;
       }
 
       const date = this.currentXScale.invert(xPosition);
-      // console.log("pointer", date, event, coords);
-
       const cumulativeCountOfWorkItems = this.currentYScale.invert(yPosition);
       const excludeCycleTime = eventName === 'scatterplot-mousemove';
 

src/graphs/moving-range/MovingRangeGraph.js

@@ -15,7 +15,6 @@ class MovingRangeGraph {
 
     // Sort the groupedArray by date to ensure correct ordering for difference calculation
     groupedArray.sort((a, b) => new Date(a.date) - new Date(b.date));
-    // console.table(groupedArray);
     // Step 3: Calculate absolute differences
     const avgLeadTimes = [];
     for (let i = 1; i < groupedArray.length; i++) {

src/graphs/scatterplot/ScatterplotGraph.js

@@ -85,7 +85,6 @@ class ScatterplotGraph {
       }
     });
     dataSet.sort((t1, t2) => t1.deliveredDate - t2.deliveredDate);
-    console.log(dataSet);
     return dataSet;
   }
 }

src/graphs/scatterplot/ScatterplotRenderer.js

@@ -40,7 +40,6 @@ class ScatterplotRenderer extends UIControlsRenderer {
    */
   constructor(data) {
     super(data);
-    console.table(data);
   }
 
   /**
@@ -134,7 +133,6 @@ class ScatterplotRenderer extends UIControlsRenderer {
     this.changeTimeInterval(false, `${this.chartName}`);
     this.drawXAxis(this.gx, this.currentXScale, this.height, true);
     this.drawYAxis(this.gy, this.currentYScale);
-    console.log(this.dotClass);
     this.chartArea
       .selectAll(`.${this.dotClass}`)
       .attr('cx', (d) => this.currentXScale(d.deliveredDate))
@@ -186,8 +184,6 @@ class ScatterplotRenderer extends UIControlsRenderer {
    * @param {d3.Scale} y - The Y-axis scale.
    */
   drawScatterplot(chartArea, data, x, y) {
-    console.log('UP___dot____', this.dotClass);
-
     chartArea
       .selectAll(`.${this.dotClass}`)
       .data(data)
@@ -212,8 +208,6 @@ class ScatterplotRenderer extends UIControlsRenderer {
   setupXAxisControl() {
     this.gx.on('click', () => {
       this.changeTimeInterval(true, `${this.chartName}`);
-      console.log(this.gx);
-      console.log(this.selectedTimeRange);
       this.drawXAxis(this.gx, this.x.copy().domain(this.selectedTimeRange), this.height, true);
     });
   }

src/graphs/scatterplot/SimpleScatterplotRenderer.js

@@ -57,7 +57,6 @@ class SimpleScatterplotRenderer extends ScatterplotRenderer {
   }
 
   drawScatterplot(chartArea, data, x, y) {
-    console.log('Simple___dot____', this.dotClass);
     chartArea
       .selectAll(`.${this.dotClass}`)
       .data(data)

src/index.js

@@ -8,7 +8,6 @@ import MovingRangeRenderer from './graphs/moving-range/MovingRangeRenderer.js';
 import ControlRenderer from './graphs/control-chart/ControlRenderer.js';
 import HistogramRenderer from './graphs/histogram/HistogramRenderer.js';
 import { eventBus } from './utils/EventBus.js';
-import { processServiceData } from './data-processor.js';
 import ObservationLoggingService from './graphs/ObservationLoggingService.js';
 
 export {
@@ -23,5 +22,4 @@ export {
   HistogramRenderer,
   ObservationLoggingService,
   eventBus,
-  processServiceData,
 };