Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multipe flightpath docking widget on logout traceback #2309

Closed
ReimarBauer opened this issue Mar 27, 2024 · 3 comments · Fixed by #2319
Closed

multipe flightpath docking widget on logout traceback #2309

ReimarBauer opened this issue Mar 27, 2024 · 3 comments · Fixed by #2319
Assignees
Labels
Milestone

Comments

@ReimarBauer
Copy link
Member

ReimarBauer commented Mar 27, 2024

when you have worked with the docking widget then closed topview and did logout.

click some operations, click a local flightpath, click again on a operation then logout

Fatal error in MSS 8.3.4 on Linux-6.5.0-10027-tuxedo-x86_64-with-glibc2.35
Python 3.11.6 | packaged by conda-forge | (main, Oct  3 2023, 10:40:35) [GCC 12.3.0]

Please report bugs in MSS to https://github.com/Open-MSS/MSS

Information about the fatal error:

Traceback (most recent call last):
  File "/home/user/PycharmProjects/MSS/mslib/msui/topview.py", line 365, in <lambda>
    self.mainwindow_signal_logout_mscolab.connect(lambda: self.signal_logout_mscolab.emit())
                                                          ^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: wrapped C/C++ object of type MSUITopViewWindow has been deleted

close_topview_then_logout

@ReimarBauer ReimarBauer added this to the 9.0.0 milestone Mar 27, 2024
@ReimarBauer ReimarBauer self-assigned this Mar 27, 2024
@ReimarBauer
Copy link
Member Author

This happens not always or a component is missing now

@matrss
Copy link
Collaborator

matrss commented Apr 4, 2024

The offending line emits the signal_logout_mscolab of the MSUITopViewWindow class whenever the signal_logout_mscolab of the main window is emitted, but there is no slot connected to that signal as far as I can tell. It also seems logically wrong that this signal on MSUITopViewWindow class exists at all. Can it just be removed?

@ReimarBauer
Copy link
Member Author

That is currently needed there.
It should logout the many operation connections in the dockingwidget when the main logout is triggered.

Some time ago I added some ideas about the docking widgets in #1108

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants