Skip to content

Fix issue 8370 #8376

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.

Sign up for GitHub

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
sklam merged 16 commits into from
Jan 11, 2023
Merged

Fix issue 8370 #8376

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
72bbc5e
added unit test to reproduce issue #8370
bszollosinagy Aug 21, 2022
6f366f5
added a unit test that reproduces another suggested problem in Issue …
bszollosinagy Aug 21, 2022
bfae727
added fix of issue #8370, as suggested by gmarkall.
bszollosinagy Aug 21, 2022
2d8ac7a
added fix for ravel() in the same spirit as the fix for issue 8370
bszollosinagy Aug 21, 2022
c24796d
added test to see if the writeable flag is set after ravel() based on…
bszollosinagy Aug 22, 2022
f3f12c6
mutability of the array resulting from ravel() is determined by wheth…
bszollosinagy Aug 22, 2022
5991a55
Streamlined a boolean expression, and added documentation string to u…
bszollosinagy Aug 22, 2022
2d70048
restart CI tests
bszollosinagy Sep 3, 2022
b43a84a
Merge branch 'main' into fix_issue_8370
bszollosinagy Sep 4, 2022
d4b40c0
only set read-only, when the source is read-only and we will not do a…
bszollosinagy Sep 4, 2022
ac22f68
added clarification about why we always remove the readonly flag when…
bszollosinagy Sep 4, 2022
84df01f
Merge branch 'numba:main' into fix_issue_8370
bszollosinagy Jan 10, 2023
61fb113
rephrased the comment about how we ensure identical behaviour with NumPy
bszollosinagy Jan 10, 2023
8f64f1b
typo: NumPy
bszollosinagy Jan 10, 2023
b1fd752
* removed the name of the exception expected when the test fails, bec…
bszollosinagy Jan 10, 2023
ea8f5e8
* now uses np.testing.assert_array_equal for more detailed error repo…
bszollosinagy Jan 11, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
added unit test to reproduce issue #8370
  • Loading branch information
@bszollosinagy
bszollosinagy committed Aug 21, 2022
commit 72bbc5e8a2fbe2d1236784aebb2c8738821c25ba
19 changes: 19 additions & 0 deletions numba/tests/test_array_manipulation.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1250,6 +1250,25 @@ def test_broadcast_arrays_incompatible_shapes_raise_valueerror(self):
self.assertIn("shape mismatch: objects cannot be broadcast to a single shape",
str(raises.exception))

def test_readonly_after_flatten(self):
# Reproduces Issue #8370
def unfold_flatten(x, y):
r, c = x.shape
a = np.broadcast_to(x, (y, r, c))
b = np.swapaxes(a, 0, 1)
cc = b.flatten()
d = np.reshape(cc, (-1, c))
d[y - 1:, :] = d[: 1 - y]
return d

pyfunc = unfold_flatten
cfunc = jit(nopython=True)(pyfunc)

# If issue #8370 is not fixed: This will fail with "NumbaTypeError:
# Cannot modify readonly array of type: readonly array(int32, 2d, C)"
Comment thread
bszollosinagy marked this conversation as resolved.
Resolved
cfunc(np.array([[1, 1, 1], [2, 2, 2], [3, 3, 3]]), 2)




if __name__ == '__main__':
Expand Down