Skip to content

Fix for errors reported in #729 when reshaping 0-elems array #756

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

Merged
merged 2 commits into from
Feb 1, 2022

Conversation

oleksandr-pavlyk
Copy link
Contributor

Fixes #729

The generated reshaped_strides routine is meant for non-empty arrays,
and so empty ones must be handled differently.

Tests added as well.

The generated reshaped_strides routine is meant for non-empty arrays,
and so empty ones must be handled differently.

Tests added as well.
@github-actions
Copy link

github-actions bot commented Feb 1, 2022

densmirn
densmirn previously approved these changes Feb 1, 2022
LukichevaPolina
LukichevaPolina previously approved these changes Feb 1, 2022
@coveralls
Copy link
Collaborator

coveralls commented Feb 1, 2022

Coverage Status

Coverage increased (+0.4%) to 81.676% when pulling 479ed60 on bugfix/gh-729-reshape into a48f381 on master.

Special case setting shape for zero-element arrays
@oleksandr-pavlyk oleksandr-pavlyk merged commit 9c90442 into master Feb 1, 2022
@oleksandr-pavlyk oleksandr-pavlyk deleted the bugfix/gh-729-reshape branch February 1, 2022 12:59
@github-actions
Copy link

github-actions bot commented Feb 1, 2022

Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. 🤞

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.

dpctl.tensor.reshape raises ValueError with empty list as input
4 participants