Always use PyQT/PySide6 for GitHub CI #23597

oscargus · 2022-08-10T13:01:51Z

PR Summary

Also, inkscape and the extra test dependencies are added to OSX so the number of skipped tests is drastically decreased.

ffmpeg can be installed, but is rather time consuming (lots of additional dependencies).

dvipng cannot be installed via brew (but basictex can and GhostScript is already installed).

PR Checklist

Tests and Styling

Has pytest style unit tests (and pytest passes).
Is Flake 8 compliant (install flake8-docstrings and run flake8 --docstring-convention=all).

Documentation

New features are documented, with examples if plot related.
New features have an entry in doc/users/next_whats_new/ (follow instructions in README.rst there).
API changes documented in doc/api/next_api_changes/ (follow instructions in README.rst there).
Documentation is sphinx and numpydoc compliant (the docs should build without error).

oscargus · 2022-08-10T16:50:37Z

lib/matplotlib/tests/test_backend_svg.py

@@ -70,6 +72,8 @@ def test_bold_font_output():
    ax.set_title('bold-title', fontweight='bold')


+@pytest.mark.skipif(sys.platform == 'darwin',
+                    reason="Fails on darwin")


This is related to the svg to png conversion. The resulting svg file is identical to a local test, but the png is slightly different.

This is the diff:

I think this will be fixed by one of the last commits in #23559 which adds quotes to the font name in the expected svg.

I think they were always mal-formed (missing quotes on the family name), but inkscape (probably actually librsvg) is getting crankier about it. At some point we fixed the bug of not putting in quotes so the rest image is correct and the expected image is wrong 🤦🏻 .

I think this will be fixed by one of the last commits in #23559 which adds quotes to the font name in the expected svg.

Seems like that didn't help. Should I simply go back to skipping (or possible better xfail-ing if that can be done on a platform-specific basis)?

xfail for now and I'll look into if I can reproduce this on a local mac.

oscargus · 2022-08-10T16:51:27Z

lib/matplotlib/tests/test_backend_qt.py

@@ -114,6 +114,7 @@ def interrupter():
        print('SUCCESS', flush=True)


+@pytest.mark.skipif(sys.platform == 'darwin', reason="Does not work on darwin")


Possibly the wrong signal is sent on OSX?

.github/workflows/tests.yml

greglucas · 2022-08-19T23:15:31Z

lib/matplotlib/tests/test_backend_qt.py

@@ -114,6 +114,7 @@ def interrupter():
        print('SUCCESS', flush=True)


+@pytest.mark.xfail(sys.platform == "darwin", reason='Fails on OSX')


I don't think this should fail on darwin? It may on CI, but I think we should put that somewhere else if possible... Especially because this will fail me locally.

Running on darwin I get:

lib/matplotlib/tests/test_backend_qt.py::test_sigint[show-kwargs0] PASSED [ 5%] lib/matplotlib/tests/test_backend_qt.py::test_sigint[pause-kwargs1] PASSED [ 7%]

I have pyqt5, version 5.15.4 installed in this env if that is any help.

You can check for 'CI' in os.environ if you think it's going to be CI-specific.

Interesting, I thought I saw these locally as well. I'll try to check tomorrow when I get home.

I think it started failing with qt6. The current CI uses qt5 and there it passes.

Now it XPASS:es, but it is not clear which qt-version is used.

XPASS lib/matplotlib/tests/test_backend_qt.py::test_sigint[pause-kwargs1] Fails on OSX XPASS lib/matplotlib/tests/test_backends_interactive.py::test_interactive_thread_safety[{'MPLBACKEND': 'qtagg', 'QT_API': 'PySide2'}] XPASS lib/matplotlib/tests/test_backends_interactive.py::test_interactive_thread_safety[{'MPLBACKEND': 'qtcairo', 'QT_API': 'PyQt5'}] XPASS lib/matplotlib/tests/test_backends_interactive.py::test_interactive_thread_safety[{'MPLBACKEND': 'qtcairo', 'QT_API': 'PyQt6'}] XPASS lib/matplotlib/tests/test_backends_interactive.py::test_interactive_thread_safety[{'MPLBACKEND': 'qtcairo', 'QT_API': 'PySide6'}] XPASS lib/matplotlib/tests/test_backends_interactive.py::test_interactive_thread_safety[{'MPLBACKEND': 'qtcairo', 'QT_API': 'PySide2'}]

I'll remove that line in the web interface and see what happens (so please squash merge...).

There seems to be some xfail that can be removed in test_interactive_thread_safety as well, but I cannot really figure out which...

lib/matplotlib/tests/test_backend_qt.py

.github/workflows/tests.yml

oscargus · 2022-08-24T11:14:42Z

So it seems to be possible to build wxPython on OSX, but quite time consuming (about an hour).

I guess it really doesn't make sense to even try to install it (still a 73 MB download) for the OSX tests. Will sort it out properly from a computer with a recent git (which this one isn't).

QuLogic · 2022-08-24T22:14:33Z

Why are we building wxPython now; there should be wheels?

oscargus · 2022-08-25T06:19:13Z

There are no wheels for OSX. Here, I was basically just trying it out, but bad idea and better to skip even trying to install wx on OSX.

(Unless we want to provide our own OSX wheel, but I have no idea how to do that. Should open an issue at them, asking for OSX wheels though since it seems to, at least, build without problems.)

.github/workflows/tests.yml

tacaswell · 2022-08-26T21:55:15Z

I moved this to 3.7 so we do not backport it. Lets keep going with the current CI on the bug-fix branch (if anything we need to freeze the packages more aggressively).

oscargus · 2022-08-28T09:35:54Z

It seems like despite it looking to be wheels for wxPython on OSX, the following happens. With:

  # But wheels for OSX
  if [[ "macOS" == 'macOS' ]]; then
    python -mpip install --upgrade wxPython &&
      python -c 'import wx' &&
      echo 'wxPython is available' ||
      echo 'wxPython is not available'
  fi

so plain "pip install wxPython"

Collecting wxPython
  Downloading wxPython-4.2.0.tar.gz (71.0 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 71.0/71.0 MB 21.7 MB/s eta 0:00:00
  Preparing metadata (setup.py): started
  Preparing metadata (setup.py): finished with status 'error'
  error: subprocess-exited-with-error
  
  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [8 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/private/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/pip-install-h25qkvuc/wxpython_a713a81ec5204ad6bcbe911a560b6c3e/setup.py", line 27, in <module>
          from buildtools.config import Config, msg, opj, runcmd, canGetSOName, getSOName
        File "/private/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/pip-install-h25qkvuc/wxpython_a713a81ec5204ad6bcbe911a560b6c3e/buildtools/config.py", line 30, in <module>
          from attrdict import AttrDict
      ModuleNotFoundError: No module named 'attrdict'
      [end of output]
  
  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
wxPython is not available

oscargus · 2022-08-28T09:41:01Z

I do not fully get why wxPython-4.2.0-cp38-cp38-macosx_11_0_universal2.whl is not deemed appropriate for 11.6.8 and Python 3.8.13 though.

oscargus · 2022-08-28T15:31:43Z

The latest version has GTK3 installed for OSX, but gives the following failing tests:

FAILED lib/matplotlib/tests/test_backends_interactive.py::test_interactive_backend[toolbar2-{'MPLBACKEND': 'gtk3cairo'}]
FAILED lib/matplotlib/tests/test_backends_interactive.py::test_interactive_backend[toolmanager-{'MPLBACKEND': 'gtk3cairo'}]
FAILED lib/matplotlib/tests/test_backends_interactive.py::test_interactive_backend[toolbar2-{'MPLBACKEND': 'gtk3agg'}]
FAILED lib/matplotlib/tests/test_backends_interactive.py::test_interactive_thread_safety[{'MPLBACKEND': 'gtk3agg'}]
FAILED lib/matplotlib/tests/test_backends_interactive.py::test_interactive_backend[toolmanager-{'MPLBACKEND': 'gtk3agg'}]
FAILED lib/matplotlib/tests/test_backends_interactive.py::test_figure_leak_20490[time_mem0-{'MPLBACKEND': 'gtk3cairo'}]
FAILED lib/matplotlib/tests/test_backends_interactive.py::test_figure_leak_20490[time_mem1-{'MPLBACKEND': 'gtk3agg'}]
FAILED lib/matplotlib/tests/test_backends_interactive.py::test_figure_leak_20490[time_mem0-{'MPLBACKEND': 'gtk3agg'}]
FAILED lib/matplotlib/tests/test_backends_interactive.py::test_blitting_events[{'MPLBACKEND': 'gtk3agg'}]
FAILED lib/matplotlib/tests/test_backends_interactive.py::test_figure_leak_20490[time_mem1-{'MPLBACKEND': 'gtk3cairo'}]

Not much info in the transcript.

tacaswell · 2022-12-16T17:34:40Z

Moved to 3.8, but if this is passing we can obviously merge it.

QuLogic · 2022-12-17T08:19:04Z

Needs a rebase, as the logs are gone, though I may have looked before and found nothing else to add.

QuLogic · 2022-12-31T22:53:59Z

It looks like there's a Traceback in stderr, but it's too long to see. Might have to run with -vvv to get pytest to print the whole thing?

jklymak · 2023-01-30T23:16:10Z

@oscargus are you going to push this through?

QuLogic · 2024-04-03T19:56:55Z

I believe this was effectively replaced by #27723.

oscargus added the topic: testing label Aug 10, 2022

oscargus force-pushed the pyqt6 branch 11 times, most recently from 2f9da0c to 77439f1 Compare August 10, 2022 16:48

oscargus commented Aug 10, 2022

View reviewed changes

oscargus force-pushed the pyqt6 branch from 77439f1 to 7a55575 Compare August 10, 2022 17:00

tacaswell added this to the v3.6.0 milestone Aug 10, 2022

oscargus force-pushed the pyqt6 branch 5 times, most recently from 69e4d77 to debf525 Compare August 15, 2022 08:39

tacaswell reviewed Aug 15, 2022

View reviewed changes

.github/workflows/tests.yml Outdated Show resolved Hide resolved

tacaswell force-pushed the pyqt6 branch from debf525 to cd81e51 Compare August 19, 2022 19:47

tacaswell approved these changes Aug 19, 2022

View reviewed changes

greglucas reviewed Aug 19, 2022

View reviewed changes

oscargus commented Aug 24, 2022

View reviewed changes

lib/matplotlib/tests/test_backend_qt.py Outdated Show resolved Hide resolved

oscargus commented Aug 24, 2022

View reviewed changes

.github/workflows/tests.yml Show resolved Hide resolved

oscargus commented Aug 26, 2022

View reviewed changes

.github/workflows/tests.yml Show resolved Hide resolved

tacaswell modified the milestones: v3.6.0, v3.7.0 Aug 26, 2022

oscargus force-pushed the pyqt6 branch from 3ab7749 to b5bc3a9 Compare August 27, 2022 09:18

oscargus force-pushed the pyqt6 branch from b5bc3a9 to 2b29502 Compare August 28, 2022 09:38

oscargus force-pushed the pyqt6 branch 3 times, most recently from 1472a35 to 81bc7cf Compare August 28, 2022 11:48

github-actions bot added the status: needs rebase label Oct 18, 2022

tacaswell modified the milestones: v3.7.0, v3.8.0 Dec 16, 2022

Always use PyQT/PySide6 for GitHub CI

d248eec

oscargus force-pushed the pyqt6 branch from 81bc7cf to d248eec Compare December 17, 2022 14:07

github-actions bot removed the status: needs rebase label Dec 17, 2022

github-actions bot added the status: needs rebase label Jan 11, 2023

jklymak marked this pull request as draft January 30, 2023 23:16

ksunden modified the milestones: v3.8.0, v3.9.0 Aug 8, 2023

QuLogic closed this Apr 3, 2024

QuLogic added the status: superseded label Apr 3, 2024

oscargus deleted the pyqt6 branch April 3, 2024 22:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Always use PyQT/PySide6 for GitHub CI #23597

Always use PyQT/PySide6 for GitHub CI #23597

oscargus commented Aug 10, 2022 •

edited

Loading

oscargus Aug 10, 2022

oscargus Aug 10, 2022

tacaswell Aug 10, 2022

oscargus Aug 14, 2022

tacaswell Aug 14, 2022

oscargus Aug 10, 2022

greglucas Aug 19, 2022

greglucas Aug 19, 2022

QuLogic Aug 20, 2022

tacaswell Aug 21, 2022

oscargus Aug 24, 2022

oscargus Aug 24, 2022

oscargus Aug 24, 2022

oscargus commented Aug 24, 2022

QuLogic commented Aug 24, 2022

oscargus commented Aug 25, 2022

tacaswell commented Aug 26, 2022

oscargus commented Aug 28, 2022

oscargus commented Aug 28, 2022

oscargus commented Aug 28, 2022

tacaswell commented Dec 16, 2022

QuLogic commented Dec 17, 2022

QuLogic commented Dec 31, 2022

jklymak commented Jan 30, 2023

QuLogic commented Apr 3, 2024

		@@ -114,6 +114,7 @@ def interrupter():
		print('SUCCESS', flush=True)


		@pytest.mark.skipif(sys.platform == 'darwin', reason="Does not work on darwin")

		@@ -114,6 +114,7 @@ def interrupter():
		print('SUCCESS', flush=True)


		@pytest.mark.xfail(sys.platform == "darwin", reason='Fails on OSX')

Always use PyQT/PySide6 for GitHub CI #23597

Always use PyQT/PySide6 for GitHub CI #23597

Conversation

oscargus commented Aug 10, 2022 • edited Loading

PR Summary

PR Checklist

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

oscargus commented Aug 24, 2022

QuLogic commented Aug 24, 2022

oscargus commented Aug 25, 2022

tacaswell commented Aug 26, 2022

oscargus commented Aug 28, 2022

oscargus commented Aug 28, 2022

oscargus commented Aug 28, 2022

tacaswell commented Dec 16, 2022

QuLogic commented Dec 17, 2022

QuLogic commented Dec 31, 2022

jklymak commented Jan 30, 2023

QuLogic commented Apr 3, 2024

oscargus commented Aug 10, 2022 •

edited

Loading