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

225 Refractor haaqi #307

Closed
wants to merge 39 commits into from
Closed

225 Refractor haaqi #307

wants to merge 39 commits into from

Conversation

groadabike
Copy link
Contributor

@groadabike groadabike commented Apr 24, 2023

Working towards a more Pythonic and faster HAAQI score.

  • Gammatone Filters in separated class
  • Ear Model in separated class
  • HAAQI Score

Close #225

Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
@groadabike groadabike added enhancement New feature or request Cadenza labels Apr 24, 2023
@groadabike groadabike self-assigned this Apr 24, 2023
@groadabike groadabike linked an issue Apr 24, 2023 that may be closed by this pull request
@groadabike
Copy link
Contributor Author

@jonbarker68, I have a GammatoneFilters class separated from HAAQI computation, where would you put it?
Also, for now, I am calling the directory clarity/evaluators/ha_metrics for lack of a better name. The idea is to put the three metrics in it.

@codecov
Copy link

codecov bot commented Apr 24, 2023

Codecov Report

Merging #307 (233dcb9) into main (d36a393) will decrease coverage by 0.43%.
The diff coverage is 90.06%.

@@            Coverage Diff             @@
##             main     #307      +/-   ##
==========================================
- Coverage   92.98%   92.55%   -0.43%     
==========================================
  Files          40       43       +3     
  Lines        3775     4418     +643     
==========================================
+ Hits         3510     4089     +579     
- Misses        265      329      +64     
Impacted Files Coverage Δ
clarity/evaluator/ha_metric/gammatone_filter.py 70.00% <70.00%> (ø)
clarity/evaluator/ha_metric/ear_model.py 88.23% <88.23%> (ø)
clarity/evaluator/ha_metric/pyhaaqi.py 94.91% <94.91%> (ø)
clarity/evaluator/haspi/eb.py 89.32% <100.00%> (ø)
clarity/utils/signal_processing.py 97.36% <100.00%> (+0.81%) ⬆️

Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
groadabike and others added 9 commits May 9, 2023 10:36
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
…-reduce-haaqi-times

# Conflicts:
#	clarity/evaluator/ha_metric/refractor_test.py
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
@groadabike groadabike changed the title 225 reduce haaqi times 225 Refractor haaqi May 12, 2023
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
Signed-off-by: Gerardo Roa Dabike <gerardo.roa@gmail.com>
@groadabike groadabike marked this pull request as ready for review May 13, 2023 13:48
@groadabike groadabike closed this May 16, 2023
@groadabike
Copy link
Contributor Author

groadabike commented May 16, 2023

PR closed to deal with the different ideas separately and in steps
Keeping the branch as reference

@groadabike groadabike deleted the 225-reduce-haaqi-times branch May 16, 2023 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cadenza enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refractoring HAAQI
1 participant