-
Notifications
You must be signed in to change notification settings - Fork 131
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
Make Python 3.7 the minimal required Python version for the worker? #2143
Comments
Only a few workers from noob are using 3.6.8 not sure how easy it is to upgrade for him |
@noobpwnftw Any comments? |
https://devguide.python.org/versions/ |
You won't hear any objections from me... |
We need to be able to contact @noobpwnftw ... |
What actually triggered this issue is that I enhanced the OpenLock library (the locking library used by the worker) with type annotations. See https://www.cantate.be/openlock for documentation. Sadly OpenLock is now no longer compatible with Python 3.6, so the latest version cannot be used by the current worker. |
I wonder when will the project grow and become serious enough to the point where it would stop waiting clients to comply instead of begging them to update. |
plot twist there is no hope for that to happen with this re-assuring behavior (on the wrong) that even copy pastes python packages inside our repo. |
Well if somebody provides a sizable fraction of the Fishtest resources he cannot be simply called a client IMHO... |
I don't understand what you are trying to say here... |
There is a simple solution, whoever wants to contribute will not stop contributing just because you want to update the code to better form. Are we really grasping at straws just so HW contributors doesn't leave, by harming the project itself? |
In principle I agree with you but I believe that it is best to handle such issues in a friendly way - if that's at all possible... |
other instances very much dependent on a few hw contributor's and their engagement with the instance.. it's just that other instances are much more shortlived or setup fairly recently, meaning they don't have any legacy worker versions to care about.. I recently came across xmake (make alternative) and saw that it is able to pull any required toolchain, which could potentially be nice to use for Stockfish, since we can manage the used toolchain without any user interactivity https://xmake.io/#/toolchain/remote_toolchain (just some thoughts) |
These days it is strongly recommended that Python packages are properly type annotated so that static type checkers such as
mypy
can catch bugs at compile time. However it is virtually impossible to work with type annotations in Python<= 3.6
- even if they are not checked.For example the crucial
from __future__ import annotations
which makes it possible to use forward references in type annotations only exists in Python>= 3.7
. So I believe it is time to move on, and raise the minimal required Python version in the worker.The text was updated successfully, but these errors were encountered: