-
-
Notifications
You must be signed in to change notification settings - Fork 473
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
convert_color_space should return copy of array #1974
Comments
Re-using the input array occurs by default when you have multiple frames to minimise the memory usage. If you had an input array with lots of frames then by creating a new array of equal size you may exceed your memory limits or result in a large slowdown (which is what happened for a couple of users, so we changed the behaviour). If your input array isn't that large you can pass If you're converting only a single frame then a new array is returned. The behaviour is a bit inconsistent and it could be better explained in the documentation. |
I have frames between 50-100 for my particular work. This is quite unexpected |
Why is it not supposed to modify the original array? You mean because the Also, as an aside, in CPython |
We can close this issue. |
Is your feature request related to a problem? Please describe.
When i use convert_color_space function, for converting YBR_FULL_422 to RGB it modifies the input original array i.e dataset.pixel_array
Describe the solution you'd like
Can it just return a copy ?
The text was updated successfully, but these errors were encountered: