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.

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

TST: add extra test case for np.array(obj, copy=False) read-only behaviour #60191

Merged
merged 6 commits into from
Nov 7, 2024
Merged

TST: add extra test case for np.array(obj, copy=False) read-only behaviour #60191

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
f895034
TST: add extra test case for np.array(obj, copy=False) read-only beha…
jorisvandenbossche Nov 5, 2024
f1e7115
fix remaining case for DataFrame raising an error if copy could not b…
jorisvandenbossche Nov 5, 2024
230180c
add test case for df copy=True
jorisvandenbossche Nov 5, 2024
31d7d72
fixup test for older numpy
jorisvandenbossche Nov 5, 2024
2183e20
Merge remote-tracking branch 'upstream/main' into array-copy-false-ex…
jorisvandenbossche Nov 7, 2024
f563fd0
fix np2 check
jorisvandenbossche Nov 7, 2024
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
Prev Previous commit
fix np2 check
  • Loading branch information
@jorisvandenbossche
jorisvandenbossche committed Nov 7, 2024
commit f563fd0ab6c5d050f6c53f3c551bfe19c98a9a48
2 changes: 1 addition & 1 deletion pandas/tests/copy_view/test_array.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@ def test_dataframe_multiple_numpy_dtypes():
assert not np.shares_memory(arr, get_array(df, "a"))
assert arr.flags.writeable is True

if not np_version_gt2:
if np_version_gt2:
# copy=False semantics are only supported in NumPy>=2.

with pytest.raises(ValueError, match="Unable to avoid copy while creating"):
Expand Down