Skip to content
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

move core 1D asserts to _check_filterbanks in ScatteringBase #876

Merged
merged 3 commits into from
Jun 18, 2022

Conversation

lostanlen
Copy link
Collaborator

Fixes #869 by @MuawizChaudhary

don't mind the title of the branch: oversampling is not becoming immutable.
also don't mind the title of the commit: _check_filterbanks belongs in ScatteringBase, not ScatteringBase1D

@lostanlen lostanlen changed the title write _check_filterbanks in ScatteringBase move core 1D asserts to _check_filterbanks in ScatteringBase Jun 12, 2022
Copy link
Collaborator

@MuawizChaudhary MuawizChaudhary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine with merging if you are @janden

@codecov-commenter
Copy link

Codecov Report

Merging #876 (4c9f8f4) into dev (88a806a) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #876      +/-   ##
==========================================
+ Coverage   87.76%   87.80%   +0.03%     
==========================================
  Files          64       64              
  Lines        2232     2238       +6     
==========================================
+ Hits         1959     1965       +6     
  Misses        273      273              
Flag Coverage Δ
jenkins_main 87.80% <100.00%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
kymatio/scattering1d/core/scattering1d.py 100.00% <ø> (ø)
kymatio/frontend/base_frontend.py 84.00% <100.00%> (+6.22%) ⬆️
kymatio/scattering1d/frontend/base_frontend.py 97.56% <100.00%> (+0.03%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88a806a...4c9f8f4. Read the comment docs.

@lostanlen lostanlen added the 1D Issue with 1D scattering code label Jun 15, 2022
@lostanlen
Copy link
Collaborator Author

@changhongw @cyrusvahidi whoever sees this first, please approve and merge

@lostanlen
Copy link
Collaborator Author

this one goes in pair with #887

Copy link
Collaborator

@cyrusvahidi cyrusvahidi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

belle

@MuawizChaudhary MuawizChaudhary merged commit 36a7af1 into kymatio:dev Jun 18, 2022
@lostanlen lostanlen deleted the immutable-oversampling branch June 25, 2022 16:23
eickenberg pushed a commit that referenced this pull request Jul 5, 2022
#876)

* write _check_filterbanks in ScatteringBase1D

* Document _check_filterbanks better
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1D Issue with 1D scattering code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants