Skip to content
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

Dask release 2024.12.0 changed dsk keyword in Blockwise class this breaks AwkwardBlockwiseLayer __init__ #557

Open
jpwgnr opened this issue Dec 4, 2024 · 5 comments

Comments

@jpwgnr
Copy link

jpwgnr commented Dec 4, 2024

Hi,
the change here in dask version 2024.12.0 breaks AwkwardBlockWiseLayer super().__init__ function:

image

I am no expert whatsoever just tried to run most recent version and saw the break in the inheritance of the class.

@martindurant
Copy link
Collaborator

@lgray , will #556 fix this? We have no way to add a dask version constraint to released versions...

@lgray
Copy link
Collaborator

lgray commented Dec 4, 2024

Yeah once I get the details worked out #556 will take care of this. For now, @jpwgnr please use an older version of dask.

@jpwgnr
Copy link
Author

jpwgnr commented Dec 4, 2024

sure all good, just wanted to let you know. :)

saw only now that you already started to fix it. Sorry for the noise.

@lgray
Copy link
Collaborator

lgray commented Dec 4, 2024

Thanks! The nightly tests in coffea also picked it up, but good to have other confirmation!

@Saransh-cpp
Copy link

+1 from vector

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants