Skip to content

393 fillers ratio bug#413

Merged
HadronCollider merged 7 commits intomasterfrom
393_FILLERS_RATIO_BUG
Apr 29, 2025
Merged

393 fillers ratio bug#413
HadronCollider merged 7 commits intomasterfrom
393_FILLERS_RATIO_BUG

Conversation

@t33nsy
Copy link
Copy Markdown
Collaborator

@t33nsy t33nsy commented Apr 18, 2025

Добавлена предобработка слов в функции get_fillers модуля app/criteria/utils - очистка от пунктуации и перевод в нижний регистр

@t33nsy t33nsy linked an issue Apr 18, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Collaborator

@HadronCollider HadronCollider left a comment

Choose a reason for hiding this comment

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

внёс изменения в авт. тесты и смежные части (чтобы заодно проверить работу на PR)

комментарии по задаче ниже

Comment thread app/criteria/utils.py Outdated
# добавлена предобработка слов - перевод в нижний регистр, очистка от пунктуации
audio_slide_words = [
recognized_word.word.value for recognized_word in audio_slide.recognized_words]
recognized_word.word.value.strip().lower().translate(str.maketrans('', '', string.punctuation))
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

чтобы не делать двойную работу (сначала убирать пробельные символы, потом пунктацию) - объедините их в maketrans

@HadronCollider HadronCollider force-pushed the 393_FILLERS_RATIO_BUG branch from d786c96 to 2af2d5e Compare April 24, 2025 14:24
@t33nsy t33nsy requested a review from HadronCollider April 25, 2025 13:45
@HadronCollider HadronCollider merged commit 82556d8 into master Apr 29, 2025
3 checks passed
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

Successfully merging this pull request may close these issues.

Баг критерия DEFAULT_FILLERS_RATIO_CRITERION

2 participants