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

[BUG] Unaccessible Second Generation of Starlet #103

Open
1 of 2 tasks
fadinammour opened this issue Sep 24, 2019 · 3 comments
Open
1 of 2 tasks

[BUG] Unaccessible Second Generation of Starlet #103

fadinammour opened this issue Sep 24, 2019 · 3 comments
Assignees
Labels
Projects

Comments

@fadinammour
Copy link

System setup
OS: macOS v10.14.5
Python version: v3.7.3
Python environment : conda v4.7.10

Describe the bug
In the C++ code, the boolean ModifiedATWT allows the user to switch from the First Generation Starlet to the Second one. However in the associated Python code, he cannot get the filters of the Second Generation because it seems that he does have access to this boolean variable.

Module and lines involved
The missing boolean variable should be an argument of get_mr_filters() in ModOpt/modoptt/signal/wavelet.py. And it can be found in Sparse2D/MR_Filter.cc and its role is explained in the comment of line 851.

Are you planning to submit a Pull Request?

  • Yes
  • No
@sfarrens sfarrens self-assigned this Sep 25, 2019
@sfarrens sfarrens added this to ToDo in PySAP-dev via automation Sep 25, 2019
@aymgal
Copy link

aymgal commented Sep 9, 2020

Hi! do you plan to add support for 2nd gen starlet ? Same question for Battle-Lemaré wavelet transform, as I know it is implemented in sparse2d (used in glimpse).

@sfarrens
Copy link
Contributor

sfarrens commented Sep 9, 2020

Hi @aymgal, I am planning to make some updates to PySAP this autumn, so I can look into.

@aymgal
Copy link

aymgal commented Sep 9, 2020

That would be awesome, thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
PySAP-dev
  
ToDo
Development

No branches or pull requests

3 participants