-
Notifications
You must be signed in to change notification settings - Fork 2
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
Let's not use Python 3.9.0 anywhere #65
Comments
This is annoying, but I think the problem is with the tasks (and then it "propagates" to the server, because by default we are using the same python version if not specified). Two comments:
EDIT: this is actually an ome-zarr-py error, which is triggered by scmultiplex. Therefore I cannot say (yet) whether it also affects fractal-tasks-core. |
In my testing, the fractal-tasks-core tasks were running fine, but the scmultiplex & 2D to 3D tasks ran into issues. 2D to 3D use the ome-zarr-py package, but scmultiplex doesn't have such a dependency. The error above is from 2D to 3D. scmultiplex runs into a similar error though. If it really is related to Python 3.9.0 / .1 issues, then the easiest way around for the moment is just not to default to use those. Thus either picking a newer 3.9 distro or (my suggestion) switching our templates to using 3.10 distributions. |
The error in ome-zarr-py comes from a dask.distributed error (see dask/distributed#7956) and maybe from somewhere even more upstream (the dask issue mentions a Python bug, likely this one, which was then fixed in 3.9.2). Thus it's not surprising that you can hit it without depending on ome-zarr-py. |
When using Python 3.9.0 to run the server, I keep running into issues like this one with tasks like scmultiplex and the 2D to 3D tasks:
I don't get those errors when the server is running on a more recent Python version (=> the tasks then also use that more recent Python version).
Thus, let's make sure we don't use any Python 3.9.0 anywhere for the server. I changed the install scripts to use Python 3.10, this issue is just to document the reasoning behind.
The text was updated successfully, but these errors were encountered: