-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
RaySession 0.13.0 hangs on Fedora 37 #152
Comments
And without debug mode:
|
Started ray-daemon manually on a fixed port and started raysession using this port and got these messages on the deamon console:
|
The problem seems to come from liblo or pyliblo. What is the python version ? |
I've got these information related to pyliblo:
And for liblo
|
hummm, I use the same versions. |
No problem with other soft using liblo ? New-Session-Manager for example, there is also carla using pyliblo, but it's not easy to use carla-control (which communicates with carla with pyliblo). |
Looking at SELinux ... Something has been changed. You can't disable SELinux temporarily. Maybe a SELinux option control the port |
Not related to SELinux. |
Looks like the inspect python module is not yet python3.11 ready ... |
RaySession doesn't use inspect module itself (normally, I sometimes use it when I am debugging if I feel really lost ;)). So it probably comes from pyliblo. pyliblo uses cython (and not python directly). That said, I know I can simply try to run it with python3.11, and see. |
Ok, I reproduce the error ! |
I found the issue. It's here : https://github.com/dsacre/pyliblo/blob/master/src/liblo.pyx#L270 I didn't know that but in this form ( var = X if condition else Y ), X is evaluated even if the condition is False, here _inspect.getargspec is evaluated, but it doesn't exists, and it raises an error. a working solution:
but continue to manage python2 is overrated now. |
Happy you were able to reproduce the bug ! |
Good new, pyliblo has already been forked, and this fork works with python3.11 . We need to ask packagers to change their base project for this python lib. |
Excellent, I will make a package on my own repo and make raysession depends on it. Thanks a lot ! |
Works fine with the pyliblo fork. |
Apparently some combination of python 3.11 and cython [0.29?] doesn't handle the one-liner correctly, causing the `X` in `X if Y else Z` to be resolved even when Y was False. I (s0600204) never had this problem, hence the "apparently". It could be that the users reporting the problem were using the release version of 0.10.0, which doesn't have the one-liner, and thus always uses the defunct `_inspect.getargspec`, causing the issue. See also: * dsacre#29 * Houston4444/RaySession#152 (comment)
I am trying to start RaySession 0.13.0 on Fedora 37 and it hangs at startup.
I started Rayssion in debug mode and got this trace:
The text was updated successfully, but these errors were encountered: