Skip to content

replace use of np.array(copy=False) with np.asarray()#313

Open
owenclaxton-SMC wants to merge 1 commit intorlabbe:masterfrom
owenclaxton-SMC:fix-multivariate-gaussian-copy
Open

replace use of np.array(copy=False) with np.asarray()#313
owenclaxton-SMC wants to merge 1 commit intorlabbe:masterfrom
owenclaxton-SMC:fix-multivariate-gaussian-copy

Conversation

@owenclaxton-SMC
Copy link

From the author's textbook, several cells fail. Example:

import kf_book.mkf_internal as mkf_internal

mean = [2., 17.]
cov = [[10., 0.], 
       [0., 4.]]

mkf_internal.plot_3d_covariance(mean, cov)

Yields:

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
...
ValueError: Unable to avoid copy while creating an array as requested.
If using `np.array(obj, copy=False)` replace it with `np.asarray(obj)` to allow a copy when needed (no behavior change in NumPy 1.x).
For more details, see https://numpy.org/devdocs/numpy_2_0_migration_guide.html#adapting-to-changes-in-the-copy-keyword.

This pull request applies this change to this function, noting the change may still be required elsewhere.

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.

1 participant