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

I found that the cost of NKF is a little bit huge. #8

Closed
c8x1 opened this issue Dec 9, 2022 · 1 comment
Closed

I found that the cost of NKF is a little bit huge. #8

c8x1 opened this issue Dec 9, 2022 · 1 comment

Comments

@c8x1
Copy link

c8x1 commented Dec 9, 2022

Although the number of parameters is small, the cost of NKF is a little bit high, especially as the number of taps increases. Have you completed the model deployment on the actual device? Do you have any ideas for simplifying the model?

@fjiang9
Copy link
Owner

fjiang9 commented Jan 29, 2023

Yes, although the computational cost of NKF is much less than other NN-based AEC models, we still think it can be furthur reduced. We plan to work on that in the near future.

@fjiang9 fjiang9 closed this as completed Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants