Skip to content

Commit ae7b845

Browse files
authored
Merge pull request #252 from githubdoe/JST/fix249
fix compass not working in 8.0.0
2 parents 9762819 + f16ac38 commit ae7b845

File tree

3 files changed

+18
-4
lines changed

3 files changed

+18
-4
lines changed

contourplot.cpp

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -492,8 +492,11 @@ ContourPlot::ContourPlot( QWidget *parent, ContourTools *tools, bool minimal ):
492492

493493
tracker_ = new MyZoomer(this->canvas(), this);
494494

495-
496-
connect(picker_, QOverload<const QPointF&>::of(&QwtPlotPicker::selected), this, &ContourPlot::selected);
495+
// Using the old SIGNAL/SLOT syntax because problems with QWT.
496+
// Qt is not able to match signal at runtime even if compile time checks all passed.
497+
// ChatGPT tells it might be an ABI problem with QWT library but I (JST) have been unable to fix for now (2025-10-20).
498+
connect(picker_, SIGNAL(selected(const QPointF&)), SLOT(selected(const QPointF&)));
499+
//connect(picker_, QOverload<const QPointF&>::of(&QwtPlotPicker::selected), this, &ContourPlot::selected);
497500

498501
QSettings settings;
499502
m_colorMapNdx = settings.value("colorMapType",0).toInt();

intensityplot.cpp

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,12 @@ intensityPlot::intensityPlot(QWidget *parent):
145145
new QwtCompassMagnetNeedle( QwtCompassMagnetNeedle::ThinStyle ) );
146146
compass->setValue( 0 );
147147
compass->setOrigin( -90 );
148-
connect(compass,&QwtAbstractSlider::valueChanged,this ,&intensityPlot::angleChanged);
148+
149+
// Using the old SIGNAL/SLOT syntax because problems with QWT.
150+
// Qt is not able to match signal at runtime even if compile time checks all passed.
151+
// ChatGPT tells it might be an ABI problem with QWT library but I (JST) have been unable to fix for now (2025-10-20).
152+
connect(compass, SIGNAL(valueChanged(double)), this ,SLOT(angleChanged(double)));
153+
//connect(compass,&QwtAbstractSlider::valueChanged,this ,&intensityPlot::angleChanged);
149154

150155
populate();
151156
resize(QGuiApplication::primaryScreen()->availableSize() * 1./ 5.);

profileplot.cpp

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,13 @@ ProfilePlot::ProfilePlot(QWidget *parent , ContourTools *tools):
198198
// new QwtCompassMagnetNeedle( QwtCompassMagnetNeedle::ThinStyle ) );
199199
compass->setValue( 270 );
200200
compass->setOrigin( -90 );
201-
connect(compass,&QwtAbstractSlider::valueChanged,this ,&ProfilePlot::angleChanged);
201+
202+
// Using the old SIGNAL/SLOT syntax because problems with QWT.
203+
// Qt is not able to match signal at runtime even if compile time checks all passed.
204+
// ChatGPT tells it might be an ABI problem with QWT library but I (JST) have been unable to fix for now (2025-10-20).
205+
connect(compass, SIGNAL(valueChanged(double)), this ,SLOT(angleChanged(double)));
206+
//connect(compass,&QwtAbstractSlider::valueChanged,this ,&ProfilePlot::angleChanged);
207+
202208
connect(m_tools, &ContourTools::newDisplayErrorRange,
203209
this, &ProfilePlot::newDisplayErrorRange);
204210
connect(m_tools, &ContourTools::contourZeroOffsetChanged, this, &ProfilePlot::zeroOffsetChanged);

0 commit comments

Comments
 (0)