Code Issues

[akhtyamovpavel]

Приветствую всех!

Комментарии, которые стоит поправить в коде:

• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/pipeline.py#L132-L142 - лучше создать словарь, в котором явно указать параметры для следующих передач в функции
• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/pipeline.py#L118-L122 - нужна конвертация в Enum: таким образом можно будет ставить меньше проверок по безопасности
• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/pipeline.py#L126-L183 - разбиваем функцию на составные части. Не должен быть код функции очень большим (Spaghetti Code)
• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/pipeline.py#L219 - data_dict точно лишнее. data - Stop Word. Dict можно будет понять по типу возвращаемого значения
• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/pipeline.py#L282 - слово from лишнее. Дополнительно стоит создать отдельный классы вида "Конвертатор"
• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/pipeline.py#L354 - функция должна быть до 40 строк. 80 строк - это перебор.
• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/pipeline.py#L399 - final, other или что-то еще? Слишком много слов без смыслов
• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/pipeline.py#L414 - нарушение Single Responsibility Principle - надо разгружать класс (разбивать на несколько классов)
• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/pipeline.py#L483 - что именно генерируем?
• https://github.com/sb-ai-lab/tsururu/blob/main/tsururu/dataset/slice.py - посмотрите, пожалуйста, аналогичные претензии. Класс, состоящий из 400 строк - это перебор

По остальным моментам - все хорошо! Чтобы стимулировать правки, сейчас поставлю 7 баллов из 10.

Буду ждать исправлений