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
multichannel to channel_axis (3 of 6): filters #5286
Conversation
df60766
to
0620227
Compare
Co-authored-by: Stefan van der Walt <sjvdwalt@gmail.com>
0620227
to
01a20b0
Compare
01a20b0
to
e4ec5b9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you Greg!
if multichannel is None: | ||
multichannel = (image.ndim == 3) | ||
|
||
multichannel = channel_axis is not None |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't we need some moveaxis logic here @grlee77 @alexdesiqueira?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
never mind, just saw that @channel_as_last_axis
takes care of that for us. Got confused because other functions don't do that. 🙏
follows #5285. Implements
channel_axis
for the filters moduleDescription
Checklist
./doc/examples
(new features only)./benchmarks
, if your changes aren't covered by anexisting benchmark
For reviewers
later.
__init__.py
.doc/release/release_dev.rst
.