silx icon indicating copy to clipboard operation
silx copied to clipboard

[test] testLineInteraction fails on windows

Open woutdenolf opened this issue 5 years ago • 9 comments
trafficstars

Windows 10, python 3.8.3, pyqt5.15 (over remote desktop)

FAIL: testLineInteraction (silx.gui.plot.tools.test.testROI.TestRegionOfInterestManager)
This test make sure that a ROI based on handles can be edited with
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\projects\silx\build\lib.win-amd64-3.8\silx\gui\plot\tools\test\testROI.py", line 570, in testLineInteraction
    self.assertNotEqual(points[0, 1], result[0, 1])
AssertionError: 0.0 == 0.0

Points and result are the same:

points: [[  0.   0.]
        [100. 100.]]
result: [[  0.   0.]
        [100. 100.]]

woutdenolf avatar Jun 05 '20 13:06 woutdenolf

Other GUI tests are flaky but this one always fails.

woutdenolf avatar Jun 05 '20 14:06 woutdenolf

It could be because of a small screen. As result there is no space to click on the plot and display the profile.

vallsv avatar Jun 05 '20 14:06 vallsv

It also fails on a Debian10 machine. I'm doing these tests remotely. The widget takes approx 1/3th of the screen size.

woutdenolf avatar Jun 08 '20 09:06 woutdenolf

Sounds weird, the screen and widget size looks to match.

vallsv avatar Jun 08 '20 10:06 vallsv

Debian10, you mean no uptodate PyQt5? What is the version you have?

vallsv avatar Jun 08 '20 10:06 vallsv

I can provide a patch to hide the profile window when a mouse interaction is needed. This way we could see if it's the problem.

Anyway, if the profile window pop over the main window and not on the side, it can't work.

vallsv avatar Jun 08 '20 10:06 vallsv

Debian10, you mean no uptodate PyQt5? What is the version you have?

pyqt 5.15

woutdenolf avatar Jun 08 '20 13:06 woutdenolf

I tried to reproduce under similar conditions. I get many errors during the tests through remote desktop on Windows... but running this test passes.

Since all tests are fine when run through CI, I would be in favor of dropping this from 0.13 milestone. Anyone with a different opinion?

t20100 avatar Jun 16 '20 14:06 t20100

I've removed the 0.13 milestone, please add it back quickly if you disagree.

t20100 avatar Jun 17 '20 09:06 t20100