Skip to content

Fix rechunking of arrays with some zero-length dimensions.#6335

Merged
TomAugspurger merged 1 commit intodask:masterfrom
Carreau:ded-48
Jun 19, 2020
Merged

Fix rechunking of arrays with some zero-length dimensions.#6335
TomAugspurger merged 1 commit intodask:masterfrom
Carreau:ded-48

Conversation

@Carreau
Copy link
Contributor

@Carreau Carreau commented Jun 19, 2020

While the arrays are technically empty and contain no values,
the rechunking code does update the chunk size metadata.

  • Tests added / passed
  • Passes black dask / flake8 dask

@Carreau
Copy link
Contributor Author

Carreau commented Jun 19, 2020

Likely "introduced" by #6233

Copy link
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix @Carreau! Overall the changes here look great, just left a couple of small comments

While the arrays are _technically_ empty and contain no values,
the rechunking code does update the chunk size metadata.
@Carreau
Copy link
Contributor Author

Carreau commented Jun 19, 2020

util function extracted, rebased and force-pushed.

@TomAugspurger
Copy link
Member

Thanks @Carreau!

@TomAugspurger TomAugspurger merged commit 4d7e674 into dask:master Jun 19, 2020
@Carreau Carreau deleted the ded-48 branch June 19, 2020 20:31
kumarprabhu1988 pushed a commit to kumarprabhu1988/dask that referenced this pull request Oct 29, 2020
While the arrays are _technically_ empty and contain no values,
the rechunking code does update the chunk size metadata.
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

Successfully merging this pull request may close these issues.

3 participants