-
Notifications
You must be signed in to change notification settings - Fork 203
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
Fix pynput import error in RQD docker #1311
Fix pynput import error in RQD docker #1311
Conversation
To include RQD conf file setup
@DiegoTavares Any thoughts here? I think you have some experience with the nimby stuff. The changes looks ok to me, at least as a stopgap. |
If I'm understanding correctly, if Nimby allows idle artist workstations to be used as RQD nodes, I was assuming that its capabilities would not be necessary for the Docker setup which will more likely be deployed in a headless environment like a cloud render farm. |
The nimby logic is set to try I agree this PR is good to prevent an error from been printed when we know pynput is not available. But it would be good to confirm not having pynput is not actually returning an error because it shouldn't |
cff3f4f
to
5fa2b49
Compare
Link the Issue(s) this Pull Request is related to.
#1301
Summarize your change.
Building and running RQD on docker results in a
ERROR:rqd.rqnimby:Failed to import pynput
caused byImportError: this platform is not supported
.This is caused by
pynput
expecting an X11 server for the GUI which docker does not have or require.Further details referenced in the pynput platform limitations doc.
This was resolved via the following steps:
USE_NIMBY_PYNPUT
changes via rqd.conf