-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Release 2023.4.0 #318
Comments
This work around dask/distributed@d62693d that @quasiben added for 2023.3.2.1 has not been replicated in main; so for next release I believe we either need to add the hack to main too or properly resolve dask/distributed#7726 ? |
Thanks for bring this up @crusaderky . @wence- has been diving quite deep into Python object handling at exit. My understanding is that he is finding the path to be extremely fragile (as many of us know) -- I'll let him chime in on dask/distributed#7726 with current thoughts but it is looking like this a much longer solution. For the short-/mid-term, we would really appreciate it if that fix could be added in Distributed, unfortunately we do not have a way to mitigate this issue another way. We will continue to explore potential solutions, but for the time being we are blocked without it. |
I do not think this is solvable before next release in a way that would be robust. I cannot yet say if I think it is possible to solve without making changes (possibly undesired) to the way resource management is done. IOW, can't give a timeline yet. |
I think Coiled would appreciate getting in dask/dask#10159. |
@galipremsagar mentioned there was a failure in |
Just following up here, my understanding is that RAPIDS pins tightly to |
That's correct for the release. If it looks like early next week the long term solution is going to take quite awhile then I'll submit PR which looks similar to the patch in 2023.3.2.1 |
Okay, looks like all blockers and best effort issues/PRs are in. I'll start pushing the release out in ~30 minutes |
@jrbourbeau Can you merge this PR after release and maybe put out a tweet to highlight it? @fjetter, @rjzamora and I have iterated on it a couple of times already but if you have any nits or other feedback please don't hesitate to push to the PR before merging. |
Yeah, no problem 👍 |
Closing as 2023.4.0 is out. Thanks all |
https://docs.dask.org/en/stable/changelog.html looks like the changelog didn't update for some reason? |
Thanks @bnaul -- things are fixed now |
Best effort
Try to close before the release but will not block the release
update_defaults
dask#10159bokeh<3
restriction dask#10177Blocker
Issues that would cause us to block and postpone the release if not fixed
grouper
-related changes dask#10182Comments
cc @fjetter @rjzamora @crusaderky @quasiben @jakirkham for visibility
Note I'm proposing we push back the release a week as most Dask folks at Coiled and several at NVIDIA will be out next Friday and the following Monday for holiday / PTO. Based on my current understanding, I don't think this should be a big deal.
The text was updated successfully, but these errors were encountered: