depthai-python icon indicating copy to clipboard operation
depthai-python copied to clipboard

Yanked PySimpleGUI 4 is now also gone

Open asperry opened this issue 1 year ago • 1 comments

A few weeks ago I submitted this issue: https://github.com/luxonis/depthai-python/issues/1042

The issue was fixed by changing PySimpleGUI to the (yanked) version 4.60.5. But alas, it appears that even the yanked version has been removed from PyPI. Therefore, running utilities/install_requirements.py is failing again.

PySimpleGUI 5 is now all that is supported and it uses a commercial licensing model.

It does appear that someone has published the last version of PySimpleGUI 4 (version 4.60.4.1) under a different name: https://pypi.org/project/PySimpleGUI-4-foss/

Perhaps using the FOSS option is the fastest fix.

asperry avatar Jul 03 '24 19:07 asperry

Thanks @asperry - @Erol444 will you roll this in?

themarpe avatar Jul 03 '24 21:07 themarpe

With the proposed fix, I'm able to install deps and run Device Manager again without any commercial license popup:

diff --git a/utilities/requirements.txt b/utilities/requirements.txt
index a1a3b72b..2fd533b1 100644
--- a/utilities/requirements.txt
+++ b/utilities/requirements.txt
@@ -4,6 +4,5 @@ numpy>=1.21.4,<3 # For RPi Buster (last successful build) and macOS M1 (first bu
 opencv-contrib-python==4.5.5.62 # Last successful RPi build, also covers M1 with above pinned numpy (otherwise 4.6.0.62 would be required, but that has a bug with charuco boards). Python version not important, abi3 wheels
 pyqt5>5,<5.15.6 ; platform_machine != "armv6l" and platform_machine != "armv7l" and platform_machine != "aarch64" and platform_machine != "arm64"
 --extra-index-url https://artifacts.luxonis.com/artifactory/luxonis-python-snapshot-local/
-# Latest version (note - it's yanked)
-PySimpleGUI==4.60.5
+PySimpleGUI-4-foss==4.60.4.1 # Mirror of final open-source release.
 depthai

mihelich avatar Jul 08 '24 20:07 mihelich

Thanks @mihelich @jakaskerl , lets roll this in

themarpe avatar Jul 12 '24 00:07 themarpe