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 (2 of 6): transform functions #5285
multichannel to channel_axis (2 of 6): transform functions #5285
Conversation
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.
@grlee77 I've made only a very minor comment, which I should have caught in the last PR — shouldn't some tests be updated to use the new form rather than wrapped in an expected_warnings context?
Co-authored-by: Stefan van der Walt <sjvdwalt@gmail.com>
ddf2ccc
to
18a5eb5
Compare
18a5eb5
to
4704517
Compare
This one has one review and should be good to go if anyone else could do a quick check |
This one is huge! Thank you Greg! |
follows #5284. implements channel_axis in the transform module
Description
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
.