-
Notifications
You must be signed in to change notification settings - Fork 127
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
Decide how to move forward with the python version #12
Comments
1+ for only maintaining one version. I tend to favour the python version if it stays a single, self contained file. I don't like to mess around with easy_install, pip or my distributions package manager to get this script running. |
I'm not a fan of superflous complexity, so yes, it will stay a single file. With your feedback, @tachylatus comment that he'd continue to contribute to a Python version in #8 and the momentum it gained with @RasmusWL's latest additions, I believe it's best to stick to the Python script. Here's what I'm going to do, unless someone objects here:
|
Sounds all reasonable to me. +1 |
Since no one complained, I went through with this. |
why? |
There are currently two versions of autorandr around due to the difficulties with #7.
The Python version has some advantages over the bash version. That is naturally the case, because in Python we can pass around objects without having to serialize everything all the time. Unique features are solutions to #8, #9, reduced configurations that only store non-default settings with autorandr.py setting the values upon loading of the profiles, support for reflection & gamma values, and of course improved speed because it does not need to call
awk
,sed
,md5sum
and so on. I therefore won't just throw that code away.On the other hand, the bash version obviously needs maintenance as well.
I'll have to decide whether to separate both versions into their own repositories, which one to promote to the version I primarily concerned myself with, and whether to choose a different name for the rewrite.
The text was updated successfully, but these errors were encountered: