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

Update some variables depending on demix_filter instead of self.algorithm. #260

Merged
merged 4 commits into from
Sep 18, 2023

Conversation

tky823
Copy link
Owner

@tky823 tky823 commented Sep 18, 2023

Summary

So far, some variables have been updated by self.algorithm as in #259. However, this lacks the expandability for customized algorithms. To address this issue, let's update them depending on the existence of self.demix_filter.

close #259

@tky823 tky823 added bss Blind source separation method bug fix Bug fixes installation installation issue labels Sep 18, 2023
@codecov-commenter
Copy link

Codecov Report

Patch coverage is 98.80% of modified lines.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Files Changed Coverage
ssspy/bss/fdica.py 66.66%
ssspy/bss/ilrma.py 100.00%
ssspy/bss/iva.py 100.00%

📢 Thoughts on this report? Let us know!.

@tky823 tky823 merged commit 471902d into main Sep 18, 2023
14 checks passed
@tky823 tky823 deleted the issue/fix-259 branch September 18, 2023 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bss Blind source separation method bug fix Bug fixes installation installation issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some procedure is not applicable in IVA
2 participants