Different logarithmic scale behavior when compared to Qwt

[nicoddemus]

Hi folks,

We are having different results when using the logarithmic scale compared with Qwt.

Qwt

from PyQt5.QtWidgets import QApplication
from Qwt5 import QwtPlot, QwtPlotCurve, QwtLog10ScaleEngine


app = QApplication([])
plot = QwtPlot()
plot.setWindowTitle("Qwt")
plot.resize(800, 600)

y = [1, 500, 1000, 1500]
x = [1, 2, 3, 4]

curve = QwtPlotCurve()
curve.setData(x, y)

engine = QwtLog10ScaleEngine()
plot.setAxisScaleEngine(QwtPlot.yLeft, engine)

curve.attach(plot)
plot.show()
app.exec()

PythonQwt

from PyQt5.QtWidgets import QApplication
from qwt import QwtPlot, QwtPlotCurve, QwtLogScaleEngine


app = QApplication([])
plot = QwtPlot()
plot.setWindowTitle("PythonQwt")
plot.resize(800, 600)

y = [1, 500, 1000, 1500]
x = [1, 2, 3, 4]

curve = QwtPlotCurve()
curve.setData(x, y)

engine = QwtLogScaleEngine()
plot.setAxisScaleEngine(QwtPlot.yLeft, engine)

curve.attach(plot)
plot.show()
app.exec()

---

The original Qwt scale looks right, but PythonQwt is not really logarithmic... any ideas what the problem might be? I suspect it might be related to the fact that now PythonQwt uses a better overall auto range calculation to give some room on each side of each axis (which looks much better in normal situations), but this extra room might not be playing well with the old logarithmic scaling code, but this is just a hunch.

I did look at the QwtLogScaleEngine.autoScale method in comparison with the original, and everything seems well.

Any hints/suggestions would be greatly appreciated! 👍

[PierreRaybaut]

After a quick overview, still no clue.
I'll have to look further into it.

[PierreRaybaut]

After some debugging, I confirm that everything works as expected in autoScale method (and in all functions and methods called in it).

I found out that this is coming from commit 3cf995f (when margin is set to None, everything works as expected).

[PierreRaybaut]

@nicoddemus : please confirm that it's fixed on your side. Should be 🤞

[nicoddemus]

Yep, I can confirm it works great, thanks a lot for this!

[PierreRaybaut]

@nicoddemus : perfect! Let me know if you need a new release soon: that would be understandable considering this bug which could be quite annoying.

[nicoddemus]

A new release soon would be great, thanks! But no rush though.