-
Notifications
You must be signed in to change notification settings - Fork 50
No ndim for iw #801
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
base: main
Are you sure you want to change the base?
No ndim for iw #801
Conversation
📚 Docs preview built and uploaded! https://www.fastplotlib.org/ver/no-ndim-for-iw |
stealing this |
I think this would also have to be changed in |
@kushalkolar Sorry I completely missed this comment! The changes I made are in init, can you elaborate? |
It's in |
@kushalkolar fixed this up |
Dont think we need ndim for image widgets. Was just using some suite2p binary files that didn't pass
_is_array_like()
because it didn't have ndim. Will test if you dont think its stupid