t.mettler/giesCANne
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix a divide by zero bug in frame info window.

Browse Source
This commit is contained in:
Collin Kidder
2019-07-17 19:25:22 -04:00
parent 017a979a1e
commit 7e212044bf
1 changed files with 35 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files
re/frameinfowindow.cpp
+7 -2
View File
@@ -373,8 +373,11 @@ void FrameInfoWindow::updateDetailsWindow(QString newID)
}
std::sort(sortedIntervals.begin(), sortedIntervals.end());
uint64_t intervalStdDiv, intervalPctl5, intervalPctl95, intervalMean, intervalVariance = 0;
uint64_t intervalStdDiv = 0, intervalPctl5 = 0, intervalPctl95 = 0, intervalMean = 0, intervalVariance = 0;
int maxTimeCounter = -1;
if (sortedIntervals.size() > 0)
{
intervalMean = intervalSum / sortedIntervals.size();
for(int l = 0; l < sortedIntervals.size(); l++) {
@@ -390,7 +393,6 @@ void FrameInfoWindow::updateDetailsWindow(QString newID)
uint64_t step = ceil((maxInterval - minInterval) / numIntervalHistBars);
int index = 0;
int counter = 0;
int maxTimeCounter = -1;
for(int l = 0; l <= numIntervalHistBars; l++) {
uint64_t currentMax = maxInterval - ((numIntervalHistBars - l) * step); // avoid missing the biggest value due to rounding errors
while(index < sortedIntervals.size()) {
@@ -406,6 +408,7 @@ void FrameInfoWindow::updateDetailsWindow(QString newID)
if(counter > maxTimeCounter) maxTimeCounter = counter;
counter = 0;
}
}
if (frameCache.count() > 1)
avgInterval = avgInterval / (frameCache.count() - 1);
@@ -519,7 +522,9 @@ void FrameInfoWindow::updateDetailsWindow(QString newID)
ui->timeHistogram->graph()->setPen(Qt::NoPen);
ui->timeHistogram->graph()->setBrush(graphBrush);
ui->timeHistogram->yAxis->setRange(0, maxTimeCounter * 1.1);
//ui->timeHistogram->xAxis->setRange(minInterval / 1000, maxInterval / 1000); //graph is in ms while intervals are in us
ui->timeHistogram->axisRect()->setupFullAxesBox();
ui->timeHistogram->rescaleAxes();
ui->timeHistogram->replot();
}
else