Skip to content

Commit ab9605c

Browse files
committed
use trace model source and correct units for graph axes and edit dialogs
1 parent 185f8a9 commit ab9605c

File tree

5 files changed

+13
-13
lines changed

5 files changed

+13
-13
lines changed

Software/PC_Application/LibreVNA-GUI/Traces/eyediagramplot.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -715,7 +715,7 @@ QString EyeDiagramPlot::mouseText(QPoint pos)
715715
auto max = qMax(abs(yAxis.getRangeMax()), abs(yAxis.getRangeMin()));
716716
auto step = abs(yAxis.getRangeMax() - yAxis.getRangeMin()) / 1000.0;
717717
significantDigits = floor(log10(max)) - floor(log10(step)) + 1;
718-
ret += Unit::ToString(coords.y(), "V", yAxis.Prefixes(), significantDigits) + "\n";
718+
ret += Unit::ToString(coords.y(), "V", yAxis.Prefixes(getModel().getSource()), significantDigits) + "\n";
719719
}
720720
return ret;
721721
}

Software/PC_Application/LibreVNA-GUI/Traces/tracewaterfall.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -284,10 +284,10 @@ void TraceWaterfall::draw(QPainter &p)
284284
}
285285
QString unit = "";
286286
if(pref.Graphs.showUnits) {
287-
unit = yAxis.Unit();
287+
unit = yAxis.Unit(getModel().getSource());
288288
}
289-
QString labelMin = Unit::ToString(yAxis.getRangeMin(), unit, yAxis.Prefixes(), 4);
290-
QString labelMax = Unit::ToString(yAxis.getRangeMax(), unit, yAxis.Prefixes(), 4);
289+
QString labelMin = Unit::ToString(yAxis.getRangeMin(), unit, yAxis.Prefixes(getModel().getSource()), 4);
290+
QString labelMax = Unit::ToString(yAxis.getRangeMax(), unit, yAxis.Prefixes(getModel().getSource()), 4);
291291
p.setPen(QPen(pref.Graphs.Color.axis, 1));
292292
p.save();
293293
p.translate(legendRect.x(), w.height());

Software/PC_Application/LibreVNA-GUI/Traces/tracexyplot.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -510,8 +510,8 @@ void TraceXYPlot::draw(QPainter &p)
510510
QString unit = "";
511511
QString prefix = " ";
512512
if(pref.Graphs.showUnits) {
513-
unit = yAxis[i].Unit();
514-
prefix = yAxis[i].Prefixes();
513+
unit = yAxis[i].Unit(getModel().getSource());
514+
prefix = yAxis[i].Prefixes(getModel().getSource());
515515
}
516516
auto tickValue = Unit::ToString(yAxis[i].getTicks()[j], unit, prefix, significantDigits);
517517
QRect bounding;
@@ -1270,7 +1270,7 @@ QString TraceXYPlot::mouseText(QPoint pos)
12701270
auto max = qMax(abs(yAxis[i].getRangeMax()), abs(yAxis[i].getRangeMin()));
12711271
auto step = abs(yAxis[i].getRangeMax() - yAxis[i].getRangeMin()) / 1000.0;
12721272
significantDigits = floor(log10(max)) - floor(log10(step)) + 1;
1273-
ret += Unit::ToString(coords[i].y(), yAxis[i].Unit(), yAxis[i].Prefixes(), significantDigits) + "\n";
1273+
ret += Unit::ToString(coords[i].y(), yAxis[i].Unit(getModel().getSource()), yAxis[i].Prefixes(getModel().getSource()), significantDigits) + "\n";
12741274
}
12751275
}
12761276
}

Software/PC_Application/LibreVNA-GUI/Traces/waterfallaxisdialog.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ WaterfallAxisDialog::WaterfallAxisDialog(TraceWaterfall *plot) :
5353
ui->Wmin->setEnabled(index != 0 && !autoRange);
5454
ui->Wmax->setEnabled(index != 0 && !autoRange);
5555
auto type = (YAxis::Type) index;
56-
QString unit = YAxis::Unit(type);
56+
QString unit = YAxis::Unit(type, plot->getModel().getSource());
5757
QString prefixes = YAxis::Prefixes(type);
5858
ui->Wmin->setUnit(unit);
5959
ui->Wmin->setPrefixes(prefixes);

Software/PC_Application/LibreVNA-GUI/Traces/xyplotaxisdialog.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ XYplotAxisDialog::XYplotAxisDialog(TraceXYPlot *plot) :
5050
}
5151

5252
// Setup GUI connections
53-
auto updateYenableState = [](QComboBox *type, QRadioButton *linear, QRadioButton *log, QCheckBox *CBauto, SIUnitEdit *min, SIUnitEdit *max, QSpinBox *divs, QCheckBox *autoDivs) {
53+
auto updateYenableState = [plot](QComboBox *type, QRadioButton *linear, QRadioButton *log, QCheckBox *CBauto, SIUnitEdit *min, SIUnitEdit *max, QSpinBox *divs, QCheckBox *autoDivs) {
5454
if(type->currentIndex() == 0) {
5555
// axis disabled
5656
log->setEnabled(false);
@@ -80,7 +80,7 @@ XYplotAxisDialog::XYplotAxisDialog(TraceXYPlot *plot) :
8080
}
8181
}
8282
auto t = (YAxis::Type) type->currentIndex();
83-
QString unit = YAxis::Unit(t);
83+
QString unit = YAxis::Unit(t, plot->getModel().getSource());
8484
QString prefixes = YAxis::Prefixes(t);
8585
min->setUnit(unit);
8686
min->setPrefixes(prefixes);
@@ -202,10 +202,10 @@ XYplotAxisDialog::XYplotAxisDialog(TraceXYPlot *plot) :
202202
ui->XautoDivs->setChecked(plot->xAxis.getAutoDivs());
203203

204204
// Constant line list handling
205-
auto editLine = [&](XYPlotConstantLine *line) {
205+
auto editLine = [plot, this](XYPlotConstantLine *line) {
206206
line->editDialog(XAxis::Unit((XAxis::Type) ui->XType->currentIndex()),
207-
YAxis::Unit((YAxis::Type) ui->Y1type->currentIndex()),
208-
YAxis::Unit((YAxis::Type) ui->Y2type->currentIndex()));
207+
YAxis::Unit((YAxis::Type) ui->Y1type->currentIndex(), plot->getModel().getSource()),
208+
YAxis::Unit((YAxis::Type) ui->Y2type->currentIndex(), plot->getModel().getSource()));
209209
};
210210

211211
for(auto l : plot->constantLines) {

0 commit comments

Comments
 (0)