DOC: Add a link to the migration guide for the deprecation warning of the copy keyword #27440
Conversation
numpy/_core/src/multiarray/ctors.c
Outdated
| "'dtype' and 'copy' keyword arguments.") < 0) { | ||
| "'dtype' and 'copy' keyword arguments. " | ||
| "To learn more, see the migration guide " | ||
| "https://numpy.org/devdocs/numpy_2_0_migration_guide.html#adapting-to-changes-in-the-copy-keyword") < 0) { |
There was a problem hiding this comment.
A 128 character line is hideous :) And there is no guarantee that it will print without breaking.
There was a problem hiding this comment.
we can remove the "#" if you want?
I didn't really know there was a migration guide until i asked #25941 (comment) and took some of your time...
There was a problem hiding this comment.
Just break the line with quotes. Python combines all quoted bits into one string.
There was a problem hiding this comment.
oh you mean hideous in the code, not hideous for the end user.
There was a problem hiding this comment.
i made the mod, but i don't think the OSX error is related right?
There was a problem hiding this comment.
The OSX error is fixed in main.
|
Thanks @hmaarrfk . |
|
thank you for the quick review! |
xref: #25941 (comment)