-
Notifications
You must be signed in to change notification settings - Fork 12
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
KeyError: 'other' when running randrctl dump #13
Comments
Thanks for the report. As an immediate workaround, please do
I'm not 100% but this likely fix the issue. I'll look into the root cause this weekend. |
Unfortunately, creating an empty yaml file didn't work ( I skimmed through the code, and it seems to assume that a key exists when it in reality doesn't here: The _configs method in the same class never seems to create the keys you are looking for, perhaps because Either way, when you create the dict maybe you should define it as I don't have time to test my suggestions right now, so I might be off, though. |
@eddie-dunn I haven't wrote this code and don't recall the details. Eventually, I was going to refactor it and eliminate ini files support in favor of yaml (since we have yaml dependency anyway). Seems that the time has come. As I said, I'll look into the issue this weekend. In case if you have a fix or an idea how to improve things, feel free to create a PR. But I do understand that the code right now is not in the best shape, so probably a lot has to be rewritten entirely. |
@eddie-dunn Thanks again for reporting! |
@edio I can confirm that your refactoring patch fixed the problem. Thanks! |
In case it matters, I installed randrctl via
pip
like so:Crash encountered in Ubuntu 18.04.
The text was updated successfully, but these errors were encountered: