Adding some code checks to avoid exceptions and reworked QMainWindow and the way main code executes

[tadeubas]

[EDITED, see all discussion below for the changes, plz accept this PR first /pull/2 ]

Hi, I've updated the project dependencies PySide and shiboken to support Python 3.7 to 3.11 (before it was like 3.5 to 3.10)

Tested and working on my end, please see if it's ok on yours too!

[pythcoiner]

thanks for this first PR!

sys.exit(app.exec_()) should be changed in sys.exit(app.exec()) here

I also notice a little change in behavior => on QLabel.setPixmap(None) here and here, the pixmap is not cleared anymore, I will open a new issue if you don't want to handle that.

[pythcoiner]

after a quick check it can be solve by this way:

    def upd_camera_stream(self, frame):
        if frame is None:
            frame = QPixmap(self.ui.video_in.size())
            frame.fill(QColor('#434343'))
            
        self.ui.video_in.setPixmap(frame)

[tadeubas]

Nice! I will do the changes

[pythcoiner]

I'm not sure about this change, what about if the soft is run on a VM (no camera), in that case, the loop will be endless

[tadeubas]

It will not, I was getting an error because I had no cameras, this made the error stop and the program execute, later I can plug the camera and the code still works

[tadeubas]

The code available_cameras.append(str(index)) on line 528 make everything to work just fine without any endless loop...

[tadeubas]

The error before this fix was (with none usbcamera):

seedqreader.py", line 522, in list_available_cameras
    if (index - int(available_cameras[-1])) > 2:
                    ~~~~~~~~~~~~~~~~~^^^^
IndexError: list index out of range

[tadeubas]

if no available camera here we got an exception and the program stops, so we need to check it before trying to access the index -1

[jdlcdl]

I recall similar and failed to report it, maybe pyqt window didn't even start. Plugged in usbcam, and solved.

[tadeubas]

Yes, it won't start without a usb camera... but now with this small fix it will start with or without a usb_cam

[pythcoiner]

if no available camera here we got an exception and the program stops, so we need to check it before trying to access the index -1

I got it but I think is better to check it there (L523)

if available_cameras and (index - int(available_cameras[-1])) > 2:

if not, even if no camera, #0 will be listed

[pythcoiner]

I add this check (L523) because, under Linux, it seems when 1 camera is plugged, I got 2 listed (only the first one available to get video stream)

[tadeubas]

I made another change to fix a problem with the terminal hanging even after closing the window because python would continue to run... now closing the window correctly ends the python execution and frees the terminal

[pythcoiner]

I made another change to fix a problem with the terminal hanging even after closing the window because python would continue to run... now closing the window correctly ends the python execution and frees the terminal

can you post a different PR for this issue (maybe also for the second one) because they are not really related to switching to Pyside6

[tadeubas]

Sure no problem, but you started asking for other changes on this PR first 😆

[pythcoiner]

Sure no problem, but you started asking for other changes on this PR first

Yes but that changes related to switching from PYside2 to PySide6 😊

[pythcoiner]

thanks!